392158 - mail prefs should have more anchors

Status: NEW

(Bugzilla :: User Interface, enhancement)

Description

[timeless]

Attachment: add more anchors

i keep wanting to direct people to various anchors, but i can't, because most of them don't exist.

Comment 1

[Vlad Dascalu]

Comment on attachment 276608 [details] [diff] [review]
add more anchors

"<a name" is used in a lot of places in the current templates (you can grep for it), and most of them aren't redundant with an "id" specification.

From my experience, most browsers understand "name" perfectly, so making it redundant with "id" seems wrong.

If you want the "id" redundancy for a good reason that I've missed, then probably that reason should apply consistently in the other template anchors as well (so maybe add "id" to them as well).

Comment 2

[Vlad Dascalu]

Comment on attachment 276608 [details] [diff] [review]
add more anchors

 [% IF Param('supportwatchers') %]
 <hr>
-<b>User Watching</b>
+<a name="user_watching" id="user_watching"><b>User Watching</b></a>

In the case above, the anchor appears only if the supportwatchers param is enabled.


+<a name="watchlist" id="watchlist"></a>
 [% IF watchedusers.size %]
 You are watching everyone in the following list:

In this case, the anchor appears independent of the fact if the user has or not someone watching.

So they are inconsistent.

A solution might be not to add those anchors at all as we don't have any guarantees for the existance of content that they should be linking to.