Closed Bug 256845 Opened 20 years ago Closed 20 years ago

Page Info commandkey is gone on Windows

Categories

(Firefox Graveyard :: Help Documentation, defect)

1.0 Branch
x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Firefox1.0beta

People

(Reporter: steffen.wilberg, Assigned: steffen.wilberg)

References

Details

Attachments

(1 file)

It used to be Ctrl+J, but that is now Downloads.
Comment on attachment 156972 [details] [diff] [review]
Don't show the Page Info row on Windows

I see no reason to show this row on Windows without having a key.
Attachment #156972 - Flags: review?(rlk)
Blocks: 253104
Comment on attachment 156972 [details] [diff] [review]
Don't show the Page Info row on Windows

>      </tr>
> -    <tr>
> +    <tr class="noWin">
>        <td>Page Info</td>
> -      <td><kbd>Ctrl</kbd>+<kbd class="win">J</kbd><kbd
> -        class="noWin">I</kbd></td>
> +      <td><kbd>Ctrl</kbd>+<kbd>I</kbd></td>
>        <td>&nbsp;</td>
>        <td><kbd>Ctrl</kbd>+<kbd>8</kbd></td>
>      </tr>

If you have noWin as the <tr>, then the hotkey will never be shown when you're
on windows, so the win is useless in the <kbd> tag.
Attachment #156972 - Flags: review?(rlk) → review-
Comment on attachment 156972 [details] [diff] [review]
Don't show the Page Info row on Windows

> so the win is useless in the <kbd> tag
That's why I removed it :)

-      <td><kbd>Ctrl</kbd>+<kbd class="win">J</kbd><kbd
-	 class="noWin">I</kbd></td>
+      <td><kbd>Ctrl</kbd>+<kbd>I</kbd></td>
Attachment #156972 - Flags: review- → review?
Comment on attachment 156972 [details] [diff] [review]
Don't show the Page Info row on Windows

See comment 4.
Attachment #156972 - Flags: review? → review?(rlk)
Comment on attachment 156972 [details] [diff] [review]
Don't show the Page Info row on Windows

oh yeah, my bad :). I should get some sleep sometime soon. r=rlk@trfenv.com
Attachment #156972 - Flags: review?(rlk) → review+
Checked in br & trunk.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox1.0beta
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: