Menu

#8 Generated HTML does not validate

open
Uzume
None
5
2021-06-20
2011-02-16
Uzume
No

HTML (and perhaps CSS too) do not validate so browsers will render in "tag soup" and/or "quirks" mode. Most of the HTML is marked with an HTML 4.01 strict doctype but much of the generated HTML does not adhere to this and fails to validate. This means it does not follow a specification and likely can and will be rendered differently in different browsers in unforeseen ways. Just because it might work for you does not mean it works for everyone. I do not think we want to do extensive browser testing so this leaves specification adherence.

Discussion

1 2 3 > >> (Page 1 of 3)
  • Uzume

    Uzume - 2011-02-16
    • summary: HTML does not validate --> Generated HTML does not validate
     
  • Uzume

    Uzume - 2011-02-19

    I checked-in work on this. So far I just removed all references of character entity references within the 128-159 (x80-x9F) range as these are defined to be extended control characters in Unicode. These were in here because Window-1252 defines these characters and it works in many situations on Windows machines anyway but this is technically invalid and can fail on some systems. The only one that needed changing was the bullet character which was misused in several places (149/x95 changed to Unicode 8226/x2022).

     
  • Uzume

    Uzume - 2011-02-19

    I checked in fixes to markup on common/login.py but this is still a work in progress.

     
  • Ahasuerus

    Ahasuerus - 2011-03-07

    r2011-09 was installed on 2011-03-06. It fixed biblio/pl.py (1.44), biblio/pubseries.py (1.6), biblio/seriesgrid.py (1.7) and common/login.py (1.7).

     
  • Uzume

    Uzume - 2013-01-06
     
  • Ahasuerus

    Ahasuerus - 2013-02-19
    • labels: 993125 -->
     
  • Ahasuerus

    Ahasuerus - 2013-02-19

    Moved to Support Requests.

     
  • Ahasuerus

    Ahasuerus - 2013-02-20

    SourceForge Patch 3269062 was integrated in biblio/tag.py 1.8 in patch r2013-53. Also see Bug 3605345, which was fixed in the same patch.

     
  • Uzume

    Uzume - 2013-03-09

    committed:
    common/login.py 1.9
    biblio/common.py 1.66
    biblio/index.py 1.9

    This is by no means complete however.

     
  • Ahasuerus

    Ahasuerus - 2013-03-13

    Partial fix of the login and main pages implemented in:

    common/login.py 1.11
    biblio/common.py 1.67
    biblio/index.py 1.11

    Installed in r2013-66 on 2013-03-13.

     
  • Ahasuerus

    Ahasuerus - 2013-03-13
    • assigned_to: nobody --> uzume
     
  • Uzume

    Uzume - 2013-03-13

    committed biblio/title.py 1.50

     
  • Ahasuerus

    Ahasuerus - 2013-05-12

    biblio/title.py 1.52 installed in r2013-72 to address a hanging "" in the Bibliographic Warnings section.

     
  • Ahasuerus

    Ahasuerus - 2013-05-15

    Added URL-escaping of ampersands to the display of links to external Web sites. Implemented in biblio/common.py 1.68, installed in r2013-73 on 2013-05-15.

     
  • Ahasuerus

    Ahasuerus - 2013-05-29

    Fixed Publication Listing in biblio/pl.py 1.59 and css/biblio_css_stub 1.4. Note that target="_blank" doesn't validate when the document type is set to "Strict", but all browsers support it. The alternative would be to use JavaScript, but we also need to re-evaluate the ability to let users decide whether they want links to external sites open in new windows. A new User Preference may be required.

     

    Last edit: Uzume 2013-06-06
  • Ahasuerus

    Ahasuerus - 2013-05-30

    Fixed author biblios in:

    biblio/biblio.py 1.25
    css/biblio_css_stub 1.5
    

    Installed in r2013-89.

     
  • Uzume

    Uzume - 2013-06-06

    I am not sure we need to place much consideration on a feature to allow external links to open in a separate viewport (windows/tab/frame, etc.). We could support such but most browsers already provide a means to open any link in a new context by just holding the shift key while clicking a link. I would rather see external links be undecorated with target= (since they do not validate in all contexts and forcing this on users goes against most usability/accessibility guidelines) and concern ourselves with more useful bibliographic features such as better web API, etc.

     
  • Ahasuerus

    Ahasuerus - 2013-06-06

    Well, it's not hard to add a User Preference to let users decide whether they want links to third party sites to open in a new window. The reason why this functionality was added a few months ago was that using Cntr-click becomes tiresome when you do it dozens or even hundreds of times a day.

     
  • Uzume

    Uzume - 2013-06-07

    Trust me--it becomes even more tiresome when it gets in your way and there is no way to turn it off. I cannot use shift or ctrl or anything else to disable it. It is almost as bad as pop-ups I cannot get rid of.

     
  • Ahasuerus

    Ahasuerus - 2013-06-07

    OK, I will bump this User Preference up on the list of priorities :-) Now to finish the current round of Title Merge changes...

     
  • Ahasuerus

    Ahasuerus - 2014-01-20

    Fixed the DTD declaration for edit scripts in edit/isfdblib.py 1.59. Installed in r2014-42 on 2014-01-20.

     
  • Ahasuerus

    Ahasuerus - 2014-03-10

    Fixed the mod header and changed ids to classes in library.py :

    biblio/common.py 1.90
    common/library.py 1.47
    edit/isfdblib.py 1.63
    mod/common.py 1.19
    mod/cv_new.py 1.15
    mod/isfdblib.py 1.18
    mod/list.py 1.26
    

    Installed in r2014-141 on 2014-03-09.

     
  • Ahasuerus

    Ahasuerus - 2014-10-07

    Replaced IDs keep, drop and label with CSS classes in:

    biblio/history.py 1.5
    common/viewers.py 1.66
    css/biblio_css_stub 1.28
    edit/pv_merge.py 1.6
    edit/verify.py 1.7
    

    Installed in r2014-243 on 2014-10-07. Keeping the SR open since there are other invalid (non-unique) IDs in the code, notably table1 and table2.

     
  • Ahasuerus

    Ahasuerus - 2015-10-11

    Changed ID to CLASS in:

    biblio/myrecent.py 1.20
    biblio/myvotes.py 1.5
    biblio/publisher.py 1.25
    biblio/pubseries.py 1.25
    biblio/recent.py 1.18
    biblio/recentver.py 1.5
    biblio/se.py 1.38
    

    Installed in r2015-154 on 2015-10-10.

     
  • Ahasuerus

    Ahasuerus - 2015-10-11

    Further ID to CLASS changes:

    biblio/top100.py 1.4
    biblio/topvoters.py 1.4
    common/viewers.py 1.99
    mod/recent.py 1.23
    

    Installed in r2015-155 on 2015-10-10.

     
1 2 3 > >> (Page 1 of 3)

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB