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

Version 80 Version 55
== Line 0 == == Line 0 ==
You should probably use the ticketing system now:
Square brackets around links to external pages look ugly. Is there any chance of preventing them from being displayed? -- [[James]]
== 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]
* 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]]
== 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]]
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 9 == == Line 8 ==
See http://dev.openguides.org/ticket/28 --[[Dom]]
* It's true that Clubs is potentially confusing as it could be used to mean University type clubs (ie societies). --[[Dom]]
== 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]]
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 15 == == Line 14 ==
See http://dev.openguides.org/ticket/12 --[[Dom]]
* Not my itch; patches welcome :) --[[Kake]]
== Line 19 == == Line 18 ==
Should new [[Comments]] be added to the top or the bottom of this page? - [[PerfDave]]
Dom: Perhaps we need specific metadata entries for URLs to other guides/reviews?
== Line 21 == == Line 20 ==
Pass. It's ugly and I'd rather we either used the mailing list or the ticketing system :) --[[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 25 ==
Should Colleges be in [[Category The University]]? --[[Dom]]
== Line 27 ==
----
== 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: How about offering the wanted pages ordered by the number of times they are wanted as well as by alphabetical ordering?
== 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]]
* Oh, nice idea. That'll require a patch to CGI::Wiki, so I'll have a think about it. --[[Kake]]
== Line 33 == == Line 30 ==
* Couldn't that be done within the stylesheet? --[[James]]
----
== Line 35 == == Line 32 ==
* Stylesheet sounds like the best plan. I'll look into it unless someone else does first. --[[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="... ?
== Line 37 == == Line 34 ==
See http://dev.openguides.org/ticket/38 --[[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 39 == == Line 36 ==
----
* 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 41 ==
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]]

== 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?
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 49 == == Line 42 ==
* Not my itch; patches welcome :) --[[Kake]]
* 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 51 == == Line 44 ==
See http://dev.openguides.org/ticket/39 --[[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]]
== Line 53 == == Line 46 ==
----
* 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 55 ==
Dom: Perhaps we need specific metadata entries for URLs to other guides/reviews?

* 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]]

== Line 49 ==
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 52 ==
<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 64 == == Line 60 ==

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]]

* 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 63 ==
----
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 67 ==
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]]
* Yes that looks nicer although I feel the html needs tweaking still. The content DIV's also appear to be identical in scope to the body of each page which feels redundant to me. Possibly there should be different div's to wrap around the content and metadata. I could e-mail you an example of my ideas if it's of any help. -- [[James]]
----
== Line 82 ==
edit_type='Normal edit'
== Line 85 ==
host='217.147.80.76'
== Line 87 ==
major_change='1'
== Line 92 ==
summary=''

Square brackets around links to external pages look ugly. Is there any chance of preventing them from being displayed? -- James

  • 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

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
  • Yes that looks nicer although I feel the html needs tweaking still. The content DIV's also appear to be identical in scope to the body of each page which feels redundant to me. Possibly there should be different div's to wrap around the content and metadata. I could e-mail you an example of my ideas if it's of any help. -- James

See also Comments Archive.


List all versions