Able to view both fake omnibox and regular omnibox in new tab page |
||||||||||
Issue descriptionApp Version: 69.0.3479.0 Canary iOS Version: 10.3.3, 11.4 Device: iPad Air, iPad pro Steps to reproduce: 1. Fresh install chrome and Launch chrome 2. Scroll NTP page slowly to the top Observed results: Notice that both omnibox are displayed Expected results: Both omnibox shouldn’t display Number of times you were able to reproduce: 5/5 Bug reproducible after clean install: Yes/No Bug reproducible after clearing cache and cookies: Yes/No Bug reproducible on Chrome Mobile on Android: Not tested Bug reproducible on Safari/Firefox: Firefox: NA, Safari: NA Bug reproducible on current stable build (App Version, iOS Version): No on M67 (UI Refresh is available from M69) Bug reproducible on the current beta channel build (App Version, iOS Version): No on M68 (UI Refresh is available from M69) Link to video: https://drive.google.com/file/d/1BAxZ2i3sdC4JK6I8GJiTV11Fn_ds2qkP/view?usp=sharing
,
Jul 5
,
Jul 6
,
Jul 6
,
Jul 8
pschaffner@ Do you want this animation changed or is this WAI?
,
Jul 10
This isn't great. justincohen@ do you think you could manage a logic that auto-scrolls the scrollview at a certain offset? To see what I mean, play with scrolling the searchbar on the root VC in Settings.app in to/out of view to see how it snaps two the two position (or see the attached video). I imagine a similar technique could be used here and somewhat alleviate the weird dual state. If this would be non-trivial to do before 69, we can change the priority to <1.
,
Jul 10
,
Jul 10
,
Jul 17
There was an error in calculating the fake omnibox scroll animation (https://chromium-review.googlesource.com/c/chromium/src/+/1138347) so the progress doesn't start until the fake omnibox starts scrolling underneath the toolbar. You can still see 2 omniboxes, but it happens much more cleanly. pschaffner@, I don't know what `auto-scrolls the scrollview at a certain offset` means or what you are suggesting from that video. I attached a video of what crrev.com/c/1138347 will look like assuming it lands as-is. WDYT?
,
Jul 17
What you've implemented looks good to me for 69. I will open a new bug for the "auto-scrolling" behavior for post-69 and try to do a better job of explaining what I want.
,
Jul 17
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/829dc7dccae21bd3e669a300b85e064551078ee9 commit 829dc7dccae21bd3e669a300b85e064551078ee9 Author: Justin Cohen <justincohen@google.com> Date: Tue Jul 17 18:36:38 2018 [ios] Image and styling tweaks to NTP. - Update most visited and search icon image. - Change hint text to go from 17pt to +.15 scale - Center hint text. - Make fake omnibox pill-like. - Tweak omnibox fade on RxR. Bug: 857433 , 854093 , 860414 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ie236f03daf20b074cb4b4382b3c70e79fc9eca9f Reviewed-on: https://chromium-review.googlesource.com/1138347 Commit-Queue: Justin Cohen <justincohen@chromium.org> Reviewed-by: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#575731} [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_bookmarks_icon.imageset/ntp_bookmarks_icon.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_bookmarks_icon.imageset/ntp_bookmarks_icon@2x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_bookmarks_icon.imageset/ntp_bookmarks_icon@3x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_history_icon.imageset/ntp_history_icon.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_history_icon.imageset/ntp_history_icon@2x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_history_icon.imageset/ntp_history_icon@3x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_readinglist_icon.imageset/ntp_readinglist_icon.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_readinglist_icon.imageset/ntp_readinglist_icon@2x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_readinglist_icon.imageset/ntp_readinglist_icon@3x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_recent_icon.imageset/ntp_recent_icon.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_recent_icon.imageset/ntp_recent_icon@2x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/cells/resources/ntp_recent_icon.imageset/ntp_recent_icon@3x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_utils.mm [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/content_suggestions_header_view.mm [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/resources/ntp_search_icon.imageset/ntp_search_icon.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/resources/ntp_search_icon.imageset/ntp_search_icon@2x.png [modify] https://crrev.com/829dc7dccae21bd3e669a300b85e064551078ee9/ios/chrome/browser/ui/content_suggestions/resources/ntp_search_icon.imageset/ntp_search_icon@3x.png
,
Jul 17
Per #c10 marking Fixed.
,
Aug 22
Behavior matches with the video attached in comment#9. Verified in iPad Pro, iOS11.4.1, M69.0.3497.53 beta |
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by rakurati@chromium.org
, Jul 5