New issue
Advanced search Search tips

Issue 856621 link

Starred by 2 users
Status: Fixed
Owner:
thegreenfrog@chromium.org
Closed: Aug 10
Cc:
mard...@chromium.org
pschaffner@chromium.org
sczs@chromium.org
martijnb@chromium.org
rohitrao@chromium.org
ios-bugs-priority@chromium.org
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug



Sign in to add a comment

Remove separator from last cell in tableview

Project Member Reported by pschaffner@chromium.org, Jun 26 2018 
This seems to already be implemented in other collections, but not Bookmarks.

Comment 1 by sczs@chromium.org, Jun 26 2018

Labels: -Pri-1 Pri-2
Owner: thegreenfrog@chromium.org
Status: Assigned (was: Untriaged)

Comment 2 by rohitrao@chromium.org, Aug 7

Labels: -Q2 -Pri-2 M-70 Pri-1
Should be easy to do, if it's already implemented for other tableviews.

Comment 3 by thegreenfrog@chromium.org, Aug 10

Status: Fixed (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 14 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/527d705f44b553625312f94ccd85bfb236f8547f

commit 527d705f44b553625312f94ccd85bfb236f8547f
Author: Chris Lu <thegreenfrog@chromium.org>
Date: Tue Aug 14 17:29:41 2018

[ios] Remove last cell separator in Bookmarks

Bug:  856621 

Change-Id: I8bae0dff5d006338a682becb8e4fdb331bc81f77
Reviewed-on: https://chromium-review.googlesource.com/1169627
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#582959}
[modify] https://crrev.com/527d705f44b553625312f94ccd85bfb236f8547f/ios/chrome/browser/ui/bookmarks/bookmark_home_view_controller.mm
►

Sign in to add a comment