The Oxford Guide - Differences between Version 80 and Version 52 of Comments

Version 80 Version 52
== Line 0 == == Line 0 ==
You should probably use the ticketing system now:
I was wondering if the Clubs category should be renamed Nightclubs, or if it's unlikely that any other sort of club will appear. :) --[[Art]]
== Line 2 == == Line 2 ==
* [http://dev.openguides.org/query?component=oxford.openguides.org Existing tickets]
* [http://dev.openguides.org/newticket?component=oxford.openguides.org New tickets]
* It's true that Clubs is potentially confusing as it could be used to mean University type clubs (ie societies). --[[Dom]]
== Line 7 == == Line 6 ==
The RSS feed for recent changes seems to be invalid, according to [http://feedvalidator.org/check.cgi?url=http%3A%2F%2Fwww.ox.compsoc.net%2Foxfordguide%2F%3Faction%3Drss this feed validator]. Apparently it's being served with the wrong Content-Type and there's confusion as to its encoding. I believe this may be causing [http://www.livejournal.com/support/see_request.bml?id=305929 this problem with LiveJournal syndication] - [[PerfDave]]
James: My spelling is bad I know. Is it possible to add some sort of spell check? Even if it's just a list of mispelled words so I can go back and re-edit the entry?
== Line 9 == == Line 8 ==
See http://dev.openguides.org/ticket/28 --[[Dom]]
* Not my itch; patches welcome :) --[[Kake]]
== Line 13 == == Line 12 ==
Can we get the edit form to insist that the user includes a comment describing their change? I keep forgetting to include something in this field. - [[PerfDave]]
Dom: Perhaps we need specific metadata entries for URLs to other guides/reviews?
== Line 15 == == Line 14 ==
See http://dev.openguides.org/ticket/12 --[[Dom]]
* Could do, yes — I mainly worry about making the edit form overcomplex and huge. But this is something the RDF people are interested in, so I'll probably add it. --[[Kake]]
== Line 19 ==
Should new [[Comments]] be added to the top or the bottom of this page? - [[PerfDave]]
== Line 21 ==
Pass. It's ugly and I'd rather we either used the mailing list or the ticketing system :) --[[Dom]]
== Line 23 == == Line 20 ==
----
James: How about offering the wanted pages ordered by the number of times they are wanted as well as by alphabetical ordering?
== Line 25 == == Line 22 ==
Should Colleges be in [[Category The University]]? --[[Dom]]
* Oh, nice idea. That'll require a patch to CGI::Wiki, so I'll have a think about it. --[[Kake]]
== Line 29 == == Line 26 ==
Square brackets around links to external pages look ugly. Is there any chance of preventing them from being displayed? -- [[James]]
James: Can the pages please be validatable HTML of some form or another? I'm trying to write a suitable stylesheet and it would make life a little easier, it's just a few loose </P> tags I believe. Should the <div id="... be <div class="... ?
== Line 31 == == Line 28 ==
* Not that I know of. I actually disagree in any case; it's a useful way of distinguishing between internal and external links, so I'd rather the convention was kept. --[[Dom]]
* Can you make me a list of the specific places where things need changing? Also, 'id' is a valid attribute for the 'div' element - it's for things that you only expect there to be one of. http://www.htmlhelp.com/reference/css/structure.html might be of help. --[[Kake]]
== Line 33 == == Line 30 ==
* Couldn't that be done within the stylesheet? --[[James]]
* I think the problem is that &lt;hr /&gt; and <&lt;form&gt; ... &lt;/form&gt; aren't valid inside paragraph tags, you can spot this on most of the pages as far as I can tell. I could be wrong, but I think that's it. Thanks for the info about the id attribute, that cleared things up for me. -- [[James]]
== Line 35 ==
* Stylesheet sounds like the best plan. I'll look into it unless someone else does first. --[[Kake]]

See http://dev.openguides.org/ticket/38 --[[Dom]]

== Line 41 == == Line 34 ==
I was wondering if the Clubs category should be renamed Nightclubs, or if it's unlikely that any other sort of club will appear. :) --[[Art]]
James: Is it nessecary to provide both a map link and the OS co-ordinates? In the current situation if streetmap etc. make a slight change to the formatting of their URL this means that every entry needs to be manually corrected, whereas if we generated the map URL from the coordinates (streetmap provide instructions at [http://www.streetmap.co.uk/linkto.htm]) only a single change would be needed to the code.
== Line 43 == == Line 36 ==
* It's true that Clubs is potentially confusing as it could be used to mean University type clubs (ie societies). --[[Dom]]
* The problem is that Streetmap have been known to be arsey in the past about software that automatically generates links to them. However, it was slightly more complicated in this particular case because the software itself was doing lookups to streetmap. The fun starts [http://london.pm.org/pipermail/london.pm/Week-of-Mon-20030203/016588.html on the london.pm mailing list]. Personally I'd be happy with doing a simple conversion as documented on their own site (I didn't notice that page before) but if Kake isn't happy with it this'll require me or someone else to maintain a patch to OpenGuides and this will need some time spent to familiarise myself with the codebase. --[[Dom]]
== Line 45 == == Line 38 ==
----
* I'm not sure how using their advertised method to generate the URL is any different (from streetmap's point of view) than having manually generated URLS linked to them. IMHO, it ''should'' be okay. The london.pm case appears to be slightly different as they were actively accessing streetmap systems to generate the information. [[James]]
== Line 47 == == Line 40 ==
James: My spelling is bad I know. Is it possible to add some sort of spell check? Even if it's just a list of mispelled words so I can go back and re-edit the entry?
* I've now added this to the Oxford installation. If a map link has been supplied, this will be linked; if not, and complete location data has been supplied, one will automatically be generated using the specification mentioned above. Patch submitted to Kake for inclusion; also available as [http://www.ox.compsoc.net/~dom/OpenGuides-automaplink.patch OpenGuides-automaplink.patch] temporarily. --[[Dom]]
== Line 49 ==
* Not my itch; patches welcome :) --[[Kake]]

See http://dev.openguides.org/ticket/39 --[[Dom]]

== Line 43 ==
I've changed the navbar to be horizontally laid-out. Is this better or worse? --[[Dom]]
* Better although a little padding between the navigation links and the search box would look a little nice, it seems a bit cramped up there. Perhaps better in the long term would be to define the navbar as something like
== Line 55 == == Line 46 ==
Dom: Perhaps we need specific metadata entries for URLs to other guides/reviews?
<pre>
&lt;div id="navbar"&gt;<br>
&lt;div class="navtool"&gt;&lt;a href="http
://www.ox.compsoc.net/oxfordguide/"&gt;Home&lt;/a&gt;&lt;/div&gt;
...<br>
&lt;div class="navtool"&gt;
... &lt;/div&gt;<br>
&lt;/div&gt;<br></pre>
which should give the designer of a style sheet more flexibility over the layout. --[[James]]

== Line 57 == == Line 54 ==
* Could do, yes &#8212; I mainly worry about making the edit form overcomplex and huge. But this is something the RDF people are interested in, so I'll probably add it. --[[Kake]]

See http://dev.openguides.org/ticket/40 --[[
Dom]]

* I've done what you suggested except that &lt;span&gt; is appropriate here, not &lt;div&gt;. I've left the pipe symbols in as separators for the time being; if someone can suggest a better way of doing this so that it'll fall back to something readable without css, let me know. --[[Dom]]
== Line 62 == == Line 56 ==



James: How
about offering the wanted pages ordered by the number of times they are wanted as well as by alphabetical ordering?

*
Oh, nice idea. That'll require a patch to CGI::Wiki, so I'll have a think about it. --[[Kake]]
*
Might also be nice to replicate [[Wanted Nodes]] as a special case in OpenGuides. --[[Dom]]
* Replicate how? --[[Kake]]
* I explained myself badly. What I mean is that Wanted Nodes should be a "special" page in OpenGuides, such that it is a place people can submit names of things they want to be added to
the guide, and it would be automatically included in the Wanted Pages page, and automatically deleted from both pages when the node exists. In fact, it's probably not that important to even have a list of the pages people have "manually" requested; it would just be a case of having a special case for the "backlink". I hope that makes slightly more sense. --[[Dom]]

Information about businesses etc needs a "last checked" field. Otherwise people won't know how reliable it is - this will particularly be a problem for the opening hours and (less so) phone numbers, from experience with another system that records this kind of information. --[[Ganesh]]
== Line 73 == == Line 58 ==
Information about businesses etc needs a "last checked" field. Otherwise people won't know how reliable it is - this will particularly be a problem for the opening hours and (less so) phone numbers, from experience with another system that records this kind of information. --[[Ganesh]]
HTML fixes: I've made an &amp; -> &amp;amp; change to fix a validation bug in the main node display (in the link to the backlinks page); the other main bug is that line breaks end up being delimited by &lt;p&gt; tags, which is invalid. This probably needs some special logic in Text::WikiFormat (maybe? haven't delved into the code enough yet). --[[Dom]]
== Line 75 ==
See http://dev.openguides.org/ticket/1 --[[Dom]]

== Line 61 ==
I've installed a stylesheet that [[James]] has contributed. I think it still needs a few tweaks, but what do other people think of it? --[[Dom]]
* I don't like the box around the front page, or the rendering of the HR's in Mozilla. -- [[James]]
* I've fixed the HRs (made them smaller) but I don't see what the different between the box on the home page is. Okay, I have just added a navbar to the home page, is that better? -- [[Dom]]
----
== Line 82 ==
edit_type='Normal edit'
== Line 85 ==
host='217.147.80.76'
== Line 87 ==
major_change='1'
== Line 92 ==
summary=''

I was wondering if the Clubs category should be renamed Nightclubs, or if it's unlikely that any other sort of club will appear. :) --Art

  • It's true that Clubs is potentially confusing as it could be used to mean University type clubs (ie societies). --Dom

James: My spelling is bad I know. Is it possible to add some sort of spell check? Even if it's just a list of mispelled words so I can go back and re-edit the entry?

  • Not my itch; patches welcome :) --Kake

Dom: Perhaps we need specific metadata entries for URLs to other guides/reviews?

  • Could do, yes — I mainly worry about making the edit form overcomplex and huge. But this is something the RDF people are interested in, so I'll probably add it. --Kake

James: How about offering the wanted pages ordered by the number of times they are wanted as well as by alphabetical ordering?

  • Oh, nice idea. That'll require a patch to CGI::Wiki, so I'll have a think about it. --Kake

James: Can the pages please be validatable HTML of some form or another? I'm trying to write a suitable stylesheet and it would make life a little easier, it's just a few loose </P> tags I believe. Should the <div id="... be <div class="... ?

  • I think the problem is that <hr /> and <<form> ... </form> aren't valid inside paragraph tags, you can spot this on most of the pages as far as I can tell. I could be wrong, but I think that's it. Thanks for the info about the id attribute, that cleared things up for me. -- James

James: Is it nessecary to provide both a map link and the OS co-ordinates? In the current situation if streetmap etc. make a slight change to the formatting of their URL this means that every entry needs to be manually corrected, whereas if we generated the map URL from the coordinates (streetmap provide instructions at http://www.streetmap.co.uk/linkto.htm) only a single change would be needed to the code.

  • The problem is that Streetmap have been known to be arsey in the past about software that automatically generates links to them. However, it was slightly more complicated in this particular case because the software itself was doing lookups to streetmap. The fun starts on the london.pm mailing list. Personally I'd be happy with doing a simple conversion as documented on their own site (I didn't notice that page before) but if Kake isn't happy with it this'll require me or someone else to maintain a patch to OpenGuides and this will need some time spent to familiarise myself with the codebase. --Dom
  • I'm not sure how using their advertised method to generate the URL is any different (from streetmap's point of view) than having manually generated URLS linked to them. IMHO, it should be okay. The london.pm case appears to be slightly different as they were actively accessing streetmap systems to generate the information. James
  • I've now added this to the Oxford installation. If a map link has been supplied, this will be linked; if not, and complete location data has been supplied, one will automatically be generated using the specification mentioned above. Patch submitted to Kake for inclusion; also available as OpenGuides-automaplink.patch temporarily. --Dom

I've changed the navbar to be horizontally laid-out. Is this better or worse? --Dom

  • Better although a little padding between the navigation links and the search box would look a little nice, it seems a bit cramped up there. Perhaps better in the long term would be to define the navbar as something like

<div id="navbar">
<div class="navtool"><a href="http://www.ox.compsoc.net/oxfordguide/">Home</a></div> ...
<div class="navtool"> ... </div>
</div>
which should give the designer of a style sheet more flexibility over the layout. --James

  • I've done what you suggested except that <span> is appropriate here, not <div>. I've left the pipe symbols in as separators for the time being; if someone can suggest a better way of doing this so that it'll fall back to something readable without css, let me know. --Dom

Information about businesses etc needs a "last checked" field. Otherwise people won't know how reliable it is - this will particularly be a problem for the opening hours and (less so) phone numbers, from experience with another system that records this kind of information. --Ganesh


HTML fixes: I've made an & -> &amp; change to fix a validation bug in the main node display (in the link to the backlinks page); the other main bug is that line breaks end up being delimited by <p> tags, which is invalid. This probably needs some special logic in Text::WikiFormat (maybe? haven't delved into the code enough yet). --Dom


I've installed a stylesheet that James has contributed. I think it still needs a few tweaks, but what do other people think of it? --Dom

  • I don't like the box around the front page, or the rendering of the HR's in Mozilla. -- James
  • I've fixed the HRs (made them smaller) but I don't see what the different between the box on the home page is. Okay, I have just added a navbar to the home page, is that better? -- Dom

See also Comments Archive.


List all versions