27410 - pref-cache.xul accesskeys need to move to .dtd

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect, P3)

Description

[Fergus Sullivan]

xpfe/components/prefwindow/resources/content/pref-cache.xul 

This file contains 3 accesskey definitions which should be moved to the 
corresponding .dtd file.

The lines in question are as follows:
<html:label for="browserCacheCheckDocFrequency0" accesskey="o" tabindex="0">
<html:label for="browserCacheCheckDocFrequency1" accesskey="e" tabindex="0">
<html:label for="browserCacheCheckDocFrequency2" accesskey="n" tabindex="0">

Comment 1

[Ben Goodger (use ben at mozilla dot org for email)]

I'll take these...

Comment 2

[Ben Goodger (use ben at mozilla dot org for email)]

fix checked in. 

Comment 3

[leger]

Bulk move of all Pref UI component bugs to new Preferences component.  Pref UI 
component will be deleted.

Comment 4

[sairuh (rarely reading bugmail)]

if verification is as easy as making sure those lines appear in the dtd file,
cool, then 'verifyme' can be removed from keywords. otherwise, if there's
additional functionality that needs testing here, i'd like some other [qa or
otherwise] help on verifying this.

Comment 5

[Fergus Sullivan]

Verification should proceed as follows:
1) Have the English UI strings been moved from the .xul file to the .dtd file?
2) Does the UI in this file actually display correctly at runtime?
3) Does the entity name in the .xul file match the entity name in the .dtd file?  
If not, the string concerned will not display correctly (and probably will not 
display at all).

Comment 6

[sairuh (rarely reading bugmail)]

thanks, fergus!

spoke with paulmac, who sez that from the perspective of this bug, steps (1) and
(3) are relevant. if (2) doesn't work, we'll file a separate bug. :-)