Closed
Bug 917617
Opened 11 years ago
Closed 11 years ago
Simulator start button UI tweaks
Categories
(DevTools Graveyard :: WebIDE, defect, P2)
DevTools Graveyard
WebIDE
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 27
People
(Reporter: jryans, Assigned: i)
Details
(Whiteboard: [good-first-bug][lang=html][mentor=paul])
Attachments
(2 files, 2 obsolete files)
41.05 KB,
image/png
|
Details | |
3.77 KB,
patch
|
paul
:
review+
|
Details | Diff | Splinter Review |
Jeff pointed out that the Simulator start button is not very clear as a "Launch" button since it is gray, and displays the internal name of the add-on without an action verb in front of it. We should use the blue color like "Connect" etc., and use a pretty name for the add-on if possible.
Comment 1•11 years ago
|
||
We should add "Start: ", exactly like in the adb helper connection section ("Connect to:").
Updated•11 years ago
|
Priority: -- → P2
Updated•11 years ago
|
Priority: P2 → P1
Updated•11 years ago
|
Priority: P1 → P2
Updated•11 years ago
|
Whiteboard: [good-first-bug][lang=html][mentor=paul]
Attachment #820397 -
Flags: review?(paul)
Hi Paul I've done some initial work on this bug, but not quite sure if it's what you want. Please take a look.
Comment 6•11 years ago
|
||
Comment on attachment 820397 [details] [diff] [review] Bug-917617-Simulator-start-button-UI-tweaks-initial-work.patch Just rename startThisSimulator to startSimulator, and then I think we can land that :) Thanks!
Attachment #820397 -
Flags: review?(paul)
There's already startSimulator been defined: http://mxr.mozilla.org/mozilla-central/source/browser/locales/en-US/chrome/browser/devtools/app-manager.dtd#45 But seems unused: http://mxr.mozilla.org/mozilla-central/search?string=connection.startSimulator&find=&findi=&filter=%5E%5B%5E%5C0%5D*%24&hitlimit=&tree=mozilla-central Did you mean ? <!ENTITY connection.startSimulator "Start Simulator"> ==> <!ENTITY connection.startSimulator "Start:"> (In reply to Paul Rouget [:paul] from comment #6) > Comment on attachment 820397 [details] [diff] [review] > Bug-917617-Simulator-start-button-UI-tweaks-initial-work.patch > > Just rename startThisSimulator to startSimulator, and then I think we can > land that :) > > Thanks!
Reporter | ||
Comment 9•11 years ago
|
||
(In reply to taj from comment #8) > I would like to work on this bug, can you assign it to me. Taj, this bug already has a patch and is nearly done. Check out the good first bug list for other ones to pick from: https://wiki.mozilla.org/DevTools/GetInvolved#Mentored_and_Good_First_Bugs
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → i
Status: NEW → ASSIGNED
Assignee | ||
Comment 10•11 years ago
|
||
Assignee | ||
Comment 11•11 years ago
|
||
(In reply to i from comment #7) > There's already startSimulator been defined: > http://mxr.mozilla.org/mozilla-central/source/browser/locales/en-US/chrome/ > browser/devtools/app-manager.dtd#45 > > But seems unused: > http://mxr.mozilla.org/mozilla-central/search?string=connection. > startSimulator&find=&findi=&filter=%5E%5B%5E%5C0%5D*%24&hitlimit=&tree=mozill > a-central > > Did you mean ? > <!ENTITY connection.startSimulator "Start Simulator"> ==> <!ENTITY > connection.startSimulator "Start:"> > > (In reply to Paul Rouget [:paul] from comment #6) > > Comment on attachment 820397 [details] [diff] [review] > > Bug-917617-Simulator-start-button-UI-tweaks-initial-work.patch > > > > Just rename startThisSimulator to startSimulator, and then I think we can > > land that :) > > > > Thanks! Oh I was wrong about this, actually connection.startSimulator was defined AND in use: http://mxr.mozilla.org/mozilla-central/source/browser/devtools/app-manager/content/connection-footer.xhtml#54 https://bug917617.bugzilla.mozilla.org/attachment.cgi?id=821416
Comment 12•11 years ago
|
||
(In reply to i from comment #11) > (In reply to i from comment #7) > > There's already startSimulator been defined: > > http://mxr.mozilla.org/mozilla-central/source/browser/locales/en-US/chrome/ > > browser/devtools/app-manager.dtd#45 > > > > But seems unused: > > http://mxr.mozilla.org/mozilla-central/search?string=connection. > > startSimulator&find=&findi=&filter=%5E%5B%5E%5C0%5D*%24&hitlimit=&tree=mozill > > a-central > > > > Did you mean ? > > <!ENTITY connection.startSimulator "Start Simulator"> ==> <!ENTITY > > connection.startSimulator "Start:"> > > > > (In reply to Paul Rouget [:paul] from comment #6) > > > Comment on attachment 820397 [details] [diff] [review] > > > Bug-917617-Simulator-start-button-UI-tweaks-initial-work.patch > > > > > > Just rename startThisSimulator to startSimulator, and then I think we can > > > land that :) > > > > > > Thanks! > > Oh I was wrong about this, > actually connection.startSimulator was defined AND in use: > > http://mxr.mozilla.org/mozilla-central/source/browser/devtools/app-manager/ > content/connection-footer.xhtml#54 > https://bug917617.bugzilla.mozilla.org/attachment.cgi?id=821416 Good catch. So I guess startThisSimulator is good.
Comment 13•11 years ago
|
||
Well… maybe not. Let's call that: `startRegisteredSimulator`.
Assignee | ||
Comment 14•11 years ago
|
||
(In reply to Paul Rouget [:paul] from comment #13) > Well… maybe not. Let's call that: `startRegisteredSimulator`. Good ideal :)
Assignee | ||
Comment 15•11 years ago
|
||
Attachment #820397 -
Attachment is obsolete: true
Attachment #821416 -
Attachment is obsolete: true
Attachment #821747 -
Flags: review?(paul)
Updated•11 years ago
|
Attachment #821747 -
Flags: review?(paul) → review+
Assignee | ||
Comment 17•11 years ago
|
||
(In reply to Paul Rouget [:paul] from comment #16) > Thank you Li! :)
Comment 18•11 years ago
|
||
Thank you for the patch! :-) https://hg.mozilla.org/integration/fx-team/rev/5615aa677b90
Keywords: checkin-needed
Assignee | ||
Comment 19•11 years ago
|
||
(In reply to Ed Morley [:edmorley UTC+1] from comment #18) > Thank you for the patch! :-) > > https://hg.mozilla.org/integration/fx-team/rev/5615aa677b90 :)
Comment 20•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5615aa677b90
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•4 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•