Uploaded image for project: 'phpBB3'
  1. phpBB3
  2. PHPBB3-9664

Another duplicate accesskey: t = top and list item

    Details

    • Type: Bug
    • Status: Unverified Fix
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0.7-PL1
    • Fix Version/s: 3.0.8-RC1
    • Component/s: Styles
    • Labels:
      None

      Description

      Since alt+t for [*] on the posting page suddenly stopped working*, I noticed that this accesskey is also assigned to <a name="top"> in overall_header.html. One of them should be changed to a different one. See also http://tracker.phpbb.com/browse/PHPBB3-8303

      *Actually after discovering the reason why it's not working anymore, I honestly wonder why it had always worked for me without any problems until now - both accesskeys haven't been changed since prosilver was released three years ago. Did someone tamper with the space time continuum recently?

        Issue Links

          Activity

          Hide
          bantu Andreas Fischer added a comment -

          Globally unused character accesskeys are only: g, v, y

          Show
          bantu Andreas Fischer added a comment - Globally unused character accesskeys are only: g, v, y
          Hide
          igorw Igor Wiedler [X] (Inactive) added a comment -

          The accesskey="t" is not working for me in firefox.

          An other issue is that your patch creates inconsistency. subsilver2 and the ACP still use t for [*]. Those would have to be adjusted too.

          Affected files:

          adm/style/acp_users_signature.html
          styles/subsilver2/template/posting_buttons.html
          

          Show
          igorw Igor Wiedler [X] (Inactive) added a comment - The accesskey="t" is not working for me in firefox. An other issue is that your patch creates inconsistency. subsilver2 and the ACP still use t for [*] . Those would have to be adjusted too. Affected files: adm/style/acp_users_signature.html styles/subsilver2/template/posting_buttons.html
          Hide
          bantu Andreas Fischer added a comment -

          Yeah, forgot the ACP.

          Show
          bantu Andreas Fischer added a comment - Yeah, forgot the ACP.

            People

            • Assignee:
              bantu Andreas Fischer
              Reporter:
              Pyramide Pyramide [X] (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development