Closed Bug 1458018 Opened 3 years ago Closed 3 years ago

Style multiselected tabs based on spec from UX

Categories

(Firefox :: Tabbed Browser, defect, P3)

Product:
Firefox
Component:
Tabbed Browser
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
Firefox 62
Tracking Flags:
Tracking Status
firefox61 --- wontfix
firefox62 --- fixed
firefox64 --- verified
firefox65 --- verified

People

(Reporter: jaws, Assigned: ablayelyfondou, Mentored)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1458018 - Add style for multiselected tabs.
59 bytes, text/x-review-board-request
jaws
: review+
Details 
UX will be providing a spec for how multiselected tabs should appear. This bug will build on the work from bug 1458010 to change from using bold tab titles to the UX-designed styling.
No longer blocks: 1458010
Depends on: 1458010
Comment on attachment 8982905 [details]
Bug 1458018 - Add style for multiselected tabs.

https://reviewboard.mozilla.org/r/248726/#review255188

::: browser/base/content/tabbrowser.xml:301
(Diff revision 1)
>            if (hoveredTab) {
>              hoveredTab._mouseenter();
>            }
> +
> +          // Update before-multiselected attributes.
> +          // gBrowser may not be inialised yet, so avoid using it

spelling, initialized (missing the 'ti' after 'in'). I searched the codebase and 'initialize' (en-US spelling) was found over 1,000 times while 'initialise' (en-GB spelling) was found 659 times.

::: browser/base/content/test/tabs/browser.ini:48
(Diff revision 1)
>  [browser_open_newtab_start_observer_notification.js]
>  [browser_bug_1387976_restore_lazy_tab_browser_muted_state.js]
>  [browser_multiselect_tabs_using_Ctrl.js]
>  [browser_multiselect_tabs_using_Shift.js]
>  [browser_multiselect_tabs_close.js]
> +[browser_multiselect_tabs_styling.js]

Can you rename this test to 'browser_multiselect_tabs_positional_attrs.js'?
Attachment #8982905 - Flags: review?(jaws) → review+
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 42dde73253debe23161e321a11610a2385c1ef03 -d 0943910146c4: rebasing 467305:42dde73253de "Bug 1458018 - Add style for multiselected tabs. r=jaws" (tip)
merging browser/base/content/tabbrowser.js
merging browser/base/content/tabbrowser.xml
merging browser/themes/shared/tabs.inc.css
warning: conflicts while merging browser/base/content/tabbrowser.js! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment on attachment 8982905 [details]
Bug 1458018 - Add style for multiselected tabs.

https://reviewboard.mozilla.org/r/248726/#review255188

Manual rebase done.
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/693da404eaca
Add style for multiselected tabs. r=jaws
https://hg.mozilla.org/mozilla-central/rev/693da404eaca
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
Depends on: 1468705
Depends on: 1472786
Verified fixed that this implementation is present on the latest Nightly Nightly 65.0a1(2018-11-22)and the latest Beta 64.0b11 as well .
Status: RESOLVED → VERIFIED
status-firefox64: --- → verified
status-firefox65: --- → verified
You need to log in before you can comment on or make changes to this bug.