Link about LEGACY extensions in Add-ons Manager is broken
Categories
(SeaMonkey :: General, defect)
Tracking
(seamonkey2.53+ fixed)
People
(Reporter: petermad, Assigned: iannbugzilla)
References
Details
(Whiteboard: SM2.53.10)
Attachments
(8 files)
67.30 KB,
image/png
|
Details | |
61.92 KB,
image/png
|
Details | |
61.21 KB,
image/png
|
Details | |
96.28 KB,
image/png
|
Details | |
60.41 KB,
image/png
|
Details | |
108.60 KB,
image/png
|
Details | |
135.40 KB,
image/png
|
Details | |
2.07 KB,
patch
|
frg
:
review+
frg
:
approval-comm-release+
frg
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:60.0) Gecko/20100101 Firefox/60.0
Steps to reproduce:
In the add-ons Manager many Extensions and Themes are marked LEGACY - I clicked the LEGACY link.
Actual results:
I am taken to https://www.seamonkey-project.org/doc/webextensions which says:
"Page Not Found
We're sorry, but we can't find what you're looking for." ... etc.
Expected results:
I expected to get information about Legacy extensions, and what it entails.
Comment 1•4 years ago
|
||
Screenshot please. Which theme? Which version and where did you get it? The legacy marking is suppressed in the distributed themes.
Comment 2•4 years ago
|
||
Bug 1365170 disabled it for all themes.
Reporter | ||
Comment 3•4 years ago
|
||
Reporter | ||
Comment 4•4 years ago
|
||
Reporter | ||
Comment 5•4 years ago
|
||
(In reply to Frank-Rainer Grahl (:frg) from comment #1)
Screenshot please. Which theme? Which version and where did you get it? The legacy marking is suppressed in the distributed themes.
All themes but SeaMonkey Default and SeaMonkey Modern ( and themes that are marked as extensions.legacy.exceptions )
All extensions except for Lightning, Chatzilla and DOM Inspector.
It is not whether or not the add-ons are marked as LEGACY or not - it is that clicking on the word LEGACY takes you to a "Page Not Fond" page, that I am describing as an error - as such not a bug in SeaMonkey, but in https://www.seamonkey-project.org
See screenshots
Comment 6•4 years ago
|
||
Yes but it seems the style change from Bug 1365170 gets overwritten or is not used by the pinball third party theme. If you use classic or modern legacy is not displayed and clicking is not possible so no error is shown.
If pinball overrides it just needs to use
.legacy-warning {
display: none;
}
and the problem will disappear.
Reporter | ||
Comment 7•4 years ago
|
||
Reporter | ||
Comment 8•4 years ago
|
||
(In reply to Frank-Rainer Grahl (:frg) from comment #6)
Yes but it seems the style change from Bug 1365170 gets overwritten or is not used by the pinball third party theme. If you use classic or modern legacy is not displayed and clicking is not possible so no error is shown.
If pinball overrides it just needs to use
.legacy-warning {
display: none;
}and the problem will disappear.
Well, see my new screenshot - I also get the legacy warning with the SeaMonkey Default Theme in Seamonkey 2.53.3 - but I will check out the css snip
Reporter | ||
Comment 9•4 years ago
|
||
Reporter | ||
Comment 10•4 years ago
|
||
Legacy warning is also shown in the shipped SeaMonkey Modern Teme - see screenshot.
The screenshot also shows a bug in the Modern themes - lines are not wrapped in description of the extensions, leading to a window in full screen width.
the css code:
.legacy-warning {
display: none;
}
works fine for Pinball - thanks for the tip.
Reporter | ||
Comment 11•4 years ago
|
||
Reporter | ||
Comment 12•4 years ago
|
||
But I would still want to know more details about legacy extensions, hence I would like the page https://www.seamonkey-project.org/doc/webextensions to give that information.
Reporter | ||
Comment 13•4 years ago
|
||
Comment 14•4 years ago
|
||
Hmm can't reproduce. Never saw the legacy link after killing it in the mentioned bug. Can you probably test in a new profile and then install pinball there to see if it changes.
As a last resort we can kill the legacy in our Mozilla Gecko release branch but probably not for 2.53.4
Updated•4 years ago
|
Reporter | ||
Comment 15•4 years ago
|
||
"Legacy" shown in stand-alon Add-ons Manger but not in tabbed Add-ons Manager
Reporter | ||
Comment 16•4 years ago
|
||
(In reply to Frank-Rainer Grahl (:frg) from comment #14)
Created attachment 9168241 [details]
Modern Theme with long lineHmm can't reproduce. Never saw the legacy link after killing it in the mentioned bug. Can you probably test in a new profile and then install pinball there to see if it changes.
As a last resort we can kill the legacy in our Mozilla Gecko release branch but probably not for 2.53.4
I tried with a new profile, and that gave me the clue: "Legacy" is shown in stand-alone Add-ons Manger but NOT in tabbed Add-ons Manager.
See screenshot
Assignee | ||
Comment 17•3 years ago
|
||
[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: Useless legacy indicator / link
Testing completed (on m-c, etc.): 2.53.10
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: none
Comment 18•3 years ago
|
||
Comment on attachment 9242940 [details] [diff] [review]
1656797-remove-legacy-25310.patch
LGTM
Comment 19•3 years ago
|
||
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/510c9f5221da
Link about LEGACY extensions in Add-ons Manager is broken. r=frg
Updated•3 years ago
|
Comment 20•3 years ago
|
||
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/51c0ad1d267ae5a9f4433fcf6b33461b1aa4190c
Link about LEGACY extensions in Add-ons Manager is broken. r=frg a=frg
Description
•