Closed Bug 1421419 Opened 7 years ago Closed 6 years ago

The X missing from search synced tabs sidebar

Categories

(Firefox :: Sync, defect, P1)

55 Branch
defect

Tracking

()

RESOLVED FIXED
Firefox 60
Tracking Status
firefox60 --- fixed

People

(Reporter: rfeeley, Assigned: eoger)

References

Details

Attachments

(2 files)

Attached image gone.png
There used to be a clear icon on the right like History and Bookmarks. Let's bring it back.
Copy/pasting irc convers:

[16:12:16]  <eoger>	they removed .compact on .search-box
[16:12:50]  <eoger>	but if you don't have .compact you don't have the css rule here: https://searchfox.org/mozilla-central/rev/9f3bd430c2b132c86c46126a0548661de876799a/browser/themes/osx/syncedtabs/sidebar.css#125

This should be an easy enough fix.
Priority: -- → P2
Assignee: nobody → eoger
Status: NEW → ASSIGNED
Priority: P2 → P1
Comment on attachment 8946414 [details]
Bug 1421419 - Fix synced tabs sidebar searchbox on macOS.

https://reviewboard.mozilla.org/r/216362/#review222288

Thanks!
Attachment #8946414 - Flags: review?(dao+bmo) → review+
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7d0ca31ba08d
Fix synced tabs sidebar searchbox on macOS. r=dao
https://hg.mozilla.org/mozilla-central/rev/7d0ca31ba08d
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
I Have tried to Reproduce the Same Bug in Firefox Nightly  60.0a1 (2018-02-20) on Windows 7 (32bit)

------------------------------------------------------------------------------------------
User Agent: Mozilla/5.0 (Windows NT 6.1; rv:60.0) Gecko/20100101 Firefox/60.0
OS: Windows_NT 6.1
Build ID: 20180220220044
Firefox Version:60.0a1
------------------------------------------------------------------------------------------

But the Bug has been Fixed. 

[Steps to Reproduce]
1) Start the Firefox with cleaned profile
2) Click view -> Sidebar -> Synced Tabs 
3) Synced tabs Sidebar will display

[Actual Results]
In the Right side of Search Synced Tabs sidebar, "X is not missing". 

Status: Fixed & Verified 

[bugday-20180221]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: