Thunderbird internal browser back/forward navigation
Categories
(Thunderbird :: Toolbars and Tabs, defect)
Tracking
(thunderbird_esr6871+ fixed, thunderbird71 fixed, thunderbird72 fixed)
People
(Reporter: projectsymphony, Assigned: Paenglab)
Details
(Whiteboard: dupme)
Attachments
(3 files, 2 obsolete files)
|
35.44 KB,
image/png
|
Details | |
|
5.36 KB,
patch
|
Paenglab
:
review+
|
Details | Diff | Splinter Review |
|
4.45 KB,
patch
|
Paenglab
:
review+
jorgk-bmo
:
approval-comm-beta+
jorgk-bmo
:
approval-comm-esr68+
|
Details | Diff | Splinter Review |
Updated•14 years ago
|
Updated•13 years ago
|
Updated•7 years ago
|
It is sad to see that no action is taken for almost a decade on reported issues.
Yes, I also feel that when browsing option is given in tb, at lease some basic browsing help should be provided, like showing address bar, Back button, forward button, back/ forward history.
I was using Thunderbrowse addon for quite some time and that provided some of these functionalities, so I didn't complain, but now that addon is abondoned, I have to request to you.
Either stop browsing within tb and open all url in default browser (ff), or if browsing is taking place within tb, some minimal set of browsing help must be there.
Thanks.
Comment 4•6 years ago
|
||
no chance to navigate away from this ...
Comment 5•6 years ago
|
||
Yes same thing here, you can even end up in a browser window that does not do anything
| Assignee | ||
Comment 6•6 years ago
|
||
Okay Geoff, this should be the right bug for this stuff.
Please if you have some spare time, could you look what is needed to work? My patch adds the buttons but they don't work yet.
Comment 7•6 years ago
|
||
I've wired this together rather differently than I had planned. Doing it this way could be considered cheating but avoids having to deal with commands and whether or not they're disabled based on the current tab etc..
The main change I made from your patch was to not use ids for the buttons, as the whole tab is cloned for each tab, and there can be only one element with any particular id. Aside from that, I am happy with your code. If you are happy with mine, let's land it.
| Assignee | ||
Comment 8•6 years ago
|
||
| Assignee | ||
Comment 9•6 years ago
|
||
This is the same patch with small changes in CSS to make the buttons theme dependent when disabled and gave some margin.
| Assignee | ||
Comment 10•6 years ago
|
||
Jörg, what do you think, should I create a patch for ESR without the tooltips? I found no appropriate replacements for them. But maybe better we have buttons without tooltips than no buttons.
| Assignee | ||
Updated•6 years ago
|
Comment 12•6 years ago
|
||
| Assignee | ||
Comment 13•6 years ago
|
||
Without tooltips for beta and ESR.
Updated•6 years ago
|
Comment 14•6 years ago
|
||
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/fb1c453eb2a0
Add navigation buttons to the content tabs. r=darktrojan
Updated•6 years ago
|
Comment 15•6 years ago
|
||
TB 71 beta 3:
https://hg.mozilla.org/releases/comm-beta/rev/f9de4d2ba542874f03c273be63666218ecd8f1d2
I thought this would be fixing bug 1577823 :-( - That's a content tab, right?
| Assignee | ||
Comment 16•6 years ago
|
||
It's a content tab but the toolbar is hidden through this rule: https://searchfox.org/comm-central/source/mail/themes/shared/mail/tabmail.css#354
But unhiding doesn't help as the buttons are always disabled.
Comment 17•6 years ago
|
||
TB 68.3 ESR:
https://hg.mozilla.org/releases/comm-esr68/rev/a942483bcdb29fc8dbf72a1234721a8b1e8cc4f9
Description
•