Last modified: 2014-09-04 04:19:47 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T71906, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69906 - 2.0: Make it clearer that Scripts and CSS must be in Gadget namespace when defining a gadget
2.0: Make it clearer that Scripts and CSS must be in Gadget namespace when de...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Gadgets (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Alex Monk
:
Depends on:
Blocks: gadgets-2.0
  Show dependency treegraph
 
Reported: 2014-08-22 15:15 UTC by Helder
Modified: 2014-09-04 04:19 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Helder 2014-08-22 15:15:00 UTC
I enabled gadgets2 role on Vagrant and when I clicked in the "Create" link in the top of
http://127.0.0.1:8080/wiki/Special:Gadgets
it loaded a popup named "Creating a new gadget". In that popup there is a field "Script" in the section "Module properties".

For a user using Gadgets 2.0 for the first time, it will not be clear what name should be typed there:
* MediaWiki:Gadget-foo.js (incorrect - this is the naming convention from 1.0)
* Gadget-foo.js (incorrect - this is the naming convention from 1.0)
* Foo.js (correct, but the page needs to exist first, or it will be ignored!)
* Gadget:Foo.js (incorrect - even if the page exists, it will be ignored)
The only case where the autocomplete feature works is when [[Gadget:Foo.js]] exists and the user types (a substring of) "Foo.js". This is not very user friendly.

Maybe there should be a placeholder as the default value, and a link to the namespace somewhere (to allow the user to find existing JS pages which can be used).

Same problems for CSS.
Comment 1 Alex Monk 2014-08-31 20:21:50 UTC
(In reply to Helder from comment #0)
> * Foo.js (correct, but the page needs to exist first, or it will be ignored!)
> * Gadget:Foo.js (incorrect - even if the page exists, it will be ignored)

I think we should accept both of these.

> Maybe there should be a placeholder as the default value, and a link to the
> namespace somewhere (to allow the user to find existing JS pages which can
> be used).

I'm thinking "Existing title in the Gadget namespace" for the placeholder. Is that enough?
Comment 2 Helder 2014-08-31 22:04:06 UTC
(In reply to Alex Monk from comment #1)
I agree.
Comment 3 Gerrit Notification Bot 2014-08-31 22:10:23 UTC
Change 157628 had a related patch set uploaded by Alex Monk:
Clarify what the 'scripts' and 'styles' fields in the gadget manager should contain

https://gerrit.wikimedia.org/r/157628
Comment 4 Gerrit Notification Bot 2014-09-04 04:04:47 UTC
Change 157628 merged by jenkins-bot:
Clarify what the 'scripts' and 'styles' fields in the gadget manager should contain

https://gerrit.wikimedia.org/r/157628

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links