Closed
Bug 342806
Opened 18 years ago
Closed 17 years ago
Suiterunner Add-ons "Install Now" button should be removed
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0a1
People
(Reporter: Callek, Assigned: prometeo.bugs)
References
Details
(Whiteboard: [good first bug])
Attachments
(1 file)
(deleted),
patch
|
standard8
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
Give a lack of "Extension/Theme Manager" component here, filing in general.
After Bug 342656 landed we now have the ability (and definately the desire) to follow suite with Firefox by hiding the button.
Should be a simple case of:
// Hides the install button in the add-ons mgr
pref("extensions.hideInstallButton", true);
In our prefs file.
Reporter | ||
Updated•18 years ago
|
Whiteboard: [good first bug]
Updated•18 years ago
|
Assignee: general → guifeatures
Component: General → XP Apps: GUI Features
QA Contact: general
Assignee | ||
Comment 1•18 years ago
|
||
Attachment #243220 -
Flags: review?(cst)
Assignee | ||
Comment 2•18 years ago
|
||
Comment on attachment 243220 [details] [diff] [review]
Is this enough?
Change of reviewer, since CTho isn't usually building suiterunner...
Attachment #243220 -
Flags: review?(cst) → review?(bugzilla)
Updated•18 years ago
|
Attachment #243220 -
Flags: review?(bugzilla) → review+
Assignee | ||
Updated•18 years ago
|
Attachment #243220 -
Flags: superreview?(neil)
Comment 3•18 years ago
|
||
Comment on attachment 243220 [details] [diff] [review]
Is this enough?
Although I'm concerned that it's not so easy to install a theme (extensions you can just open in the browser).
Attachment #243220 -
Flags: superreview?(neil) → superreview+
Assignee | ||
Comment 4•18 years ago
|
||
(In reply to comment #3)
> (From update of attachment 243220 [details] [diff] [review] [edit])
> Although I'm concerned that it's not so easy to install a theme (extensions you
> can just open in the browser).
I agree with Neil: it's much more logical to show the install button, and refuse to install an xpi by Open file..., so that addons are logically enforced to be managed by the addon manager only.
Just my €.02...
Reporter | ||
Comment 5•17 years ago
|
||
Hmm, seems to have slipped through the cracks; Still wanted...
I haven't tested that this still applies, so if there is a conflict here I'll submit an updated patch.
Keywords: checkin-needed
Updated•17 years ago
|
Assignee: guifeatures → prometeo.bugs
QA Contact: guifeatures
Comment 6•17 years ago
|
||
Checking in suite/browser/browser-prefs.js;
/cvsroot/mozilla/suite/browser/browser-prefs.js,v <-- browser-prefs.js
new revision: 1.69; previous revision: 1.68
done
Status: NEW → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Summary: Suiterunner Add-ons "Install Now" button should be removed. → Suiterunner Add-ons "Install Now" button should be removed
Comment 7•17 years ago
|
||
Sorry, but I don't see _any_ justification why this button should be hidden?
This is just plain annoying, especially since it doesn't solve any problem or something...
Reporter | ||
Comment 8•17 years ago
|
||
because it is intended to be shown for apps without the built-in UI to *browse* to a site which contains the addon. We Do.
I don't think its worth exposing the install now as 99.99% of users will not be *downloading* an xpi then trying to install it; rather they will be browsing the web, addons.m.o for example, and click to install; getting the regular UI.
The button was needed for Thunderbird to have a discoverable way to install addons, in SeaMonkey I don't feel the few who would use it/want it warrants having the pref flipped by default (or exposed in pref UI).
Comment 9•17 years ago
|
||
Having an Addon-Manager which can't install addons is just weird.
- It's inconsistent, non-obvious and counter-intuitive.
- It makes "download once, install offline/often" very hard and non-obvious. Many users (as seen in newsgroups and mails I got) maintain SM based installations for their family/friends and are downloading addons on memory sticks etc. to install them offline, on several mechines. Our capability of browsing the local file system in the browser is very obscure and unwieldy (in short: user-unfriendly).
Hiding the button doesn't solve any problems, it's just taking away a useful and logical way of installing extensions...
I think we should back out this change.
You need to log in
before you can comment on or make changes to this bug.
Description
•