Issue metadata
Sign in to add a comment
|
Update Find-in-page bar to Material Design (Mac) |
||||||||||||||||||||||
Issue descriptionHere is the spec (and note that icon colors, etc. probably have to flip for Incognito, but are also overridden by the theme).
,
Feb 25 2017
It might make sense to just do this in Views and ship it with #md-secondary-ui
,
Feb 28 2017
Ooh interesting - I didn't realise this is so close in appearance to a floating window/bubble. So, it does look possible to do this. The current MD window/animation does compile and "work" on Mac, but it's pretty janky (see attached). Issues - framerate could be better (partly due to debug build maybe) - appears to "pop" in rather than slide (again, partly due to using a debug build in this video, but there's something to optimize/waitfor here) - when sliding out, the contents move out over a white background (making the background transparent should fix this, but might have shadow implications) - sometimes the WindowServer-drawn window shadow doesn't get invalidated, or doesn't get drawn (I couldn't repro that in this video, but it can happen). Also plumbing this through for a Cocoa browser could get curly because of possible assumptions around whether the findbar is an NSView versus and NSWindow.
,
Mar 20 2017
,
Apr 12 2017
,
May 25 2017
,
May 25 2017
,
Jun 16 2017
,
Aug 25 2017
,
Oct 16 2017
This is waiting on macviews secondary UI, but doesn't need to block Harmony launch, so I'm tagging M-X.
,
Jan 4 2018
Since we are using the NSView findbar for Harmony Phase 1, this should perhaps be labelled MacViews-Browser. Fixing it separately is not on the agenda AFAIK, so I'm dropping priority. It also basically works already: See Issue 646734 . The *Harmony* bug for this on non-Mac is Issue 651643 , and it is mostly done. Unless we are planning to replace the NSView findbar before MacViewsBrowser, I don't think we need to keep this bug open.
,
Mar 12 2018
,
Mar 21 2018
This needs a milestone attached to it. We can't have this UI for 69. And I assume we don't want it for MacViews in 67?
,
Mar 21 2018
This is going to be tied to MacViews-Browser shipping, which is targeted at M69. Once MacViews-Browser ships, this will happen automatically. Does that work, bettes@? Getting the Views find bar to work with the Cocoa browser window (pre-M69) or getting the Cocoa find bar to look like this would both be undertakings.
,
Mar 21 2018
,
Mar 23 2018
MacViews triage: marking this as Target-69 and assigning to myself (to ensure that that happens).
,
Mar 27 2018
,
Jun 21 2018
Issue 48568 has been merged into this issue.
,
Jul 12
,
Jul 12
,
Jul 21
,
Jul 23
Per c#14, this is really a Cocoa thing.
,
Aug 1
Can we close this out with MacViews?
,
Aug 2
yeah I think we can WontFix everything in Hotlist-CocoaBrowser in bulk. (at some point). (if things go to plan). Although there's a lot more.. like, 3 of the bugs blocked on this one that we can probably also close out.
,
Aug 2
Punting Hotlist-CocoaBrowser to M71, at which point I will mass-close them. NextAction is the M71 branch point.
,
Oct 11
The NextAction date has arrived: 2018-10-11
,
Oct 11
Hotlist-CocoaBrowser -> Closed :) |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by lgrey@chromium.org
, Feb 24 2017