[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #21234 [Metrics/Atlas]: Fix W3C validator issues



#21234: Fix W3C validator issues
---------------------------+--------------------------------
 Reporter:  cypherpunks    |          Owner:  irl
     Type:  defect         |         Status:  needs_revision
 Priority:  Medium         |      Milestone:
Component:  Metrics/Atlas  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+--------------------------------
Changes (by irl):

 * status:  needs_review => needs_revision


Comment:

 Thanks for the patch cypherpunks, but I don't think this can be merged.

 We do kinda need an action there. The reason it's empty is because it's
 for the same page. Apparently browsers can cope with this, and
 semantically action="" is the same as just action, which is how this
 worked before. I think removing it would not be the best course to take as
 this would change (potentially, but I'm not a browser expert) the semantic
 meaning.

 If it needs to be non-empty, perhaps action="#" is the answer we're
 looking for. I'll test that out when I get some time, if you have time
 beforehand to check then it would be appreciated.

 (You may also be interested in #21274)

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21234#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs