Closed Bug 993973 Opened 11 years ago Closed 11 years ago

Move the View menu after the Edit menu

Categories

(DevTools Graveyard :: Scratchpad, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 31

People

(Reporter: past, Assigned: beberveiga)

References

Details

(Whiteboard: [good first bug][mentor=past@mozilla.com][lang=js])

Attachments

(1 file)

Bug 953206 introduced a View menu for Scratchpad, but placed it before the Edit menu. That is not following the traditional ordering of menus, so we should move View after Edit. Swapping the <menu> tags in scratchpad.xul should be all that is needed to accomplish this.
Sir , i would like to fix this bug :)
Comment on attachment 8404262 [details] [diff] [review] 993973-move-the-view-menu-after-the-edit-menu-v1.patch Review of attachment 8404262 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, thanks for the patch!
Attachment #8404262 - Flags: review?(past) → review+
Assignee: nobody → contact
Status: NEW → ASSIGNED
Whiteboard: [good first bug][mentor=past@mozilla.com][lang=js] → [good first bug][mentor=past@mozilla.com][lang=js][fixed-in-fx-team]
vikneshwar, please keep in mind that you don't need anyone's permission to work on a bug! Here is a list of unowned, simple devtools bugs that you might be interested in: http://www.joshmatthews.net/bugsahoy/?devtools=1&unowned=1&simple=1
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug][mentor=past@mozilla.com][lang=js][fixed-in-fx-team] → [good first bug][mentor=past@mozilla.com][lang=js]
Target Milestone: --- → Firefox 31
Verified Fixed on Win 8 64 bit Firefox 31.0a1(2014-04-15)
Status: RESOLVED → VERIFIED
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: