Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

NTP site removal notification interface needs improvements #8097

Closed
cezaraugusto opened this issue Feb 6, 2020 · 2 comments
Closed

NTP site removal notification interface needs improvements #8097

cezaraugusto opened this issue Feb 6, 2020 · 2 comments
Labels
closed/duplicate Issue has already been reported feature/new-tab regression

Comments

@cezaraugusto
Copy link
Contributor

Position doesn't look good, currently overflows stats and words are too close.

Screen Shot 2020-02-06 at 10 49 38 AM

Introduced with https://github.com/brave/brave-core/pull/4075/files#diff-4f7fe7112dc0ec2b5cd45ecb5f76c0cdR100-R104.

Labeling as regression since it deviates from previous versions interface. Please change if I'm wrong.

cc @petemill @karenkliu
@rebron
Copy link
Collaborator

rebron commented Apr 3, 2020

Closing. This was fixed with an earlier change. #8282

@rebron rebron closed this as completed Apr 3, 2020
@rebron rebron added the closed/duplicate Issue has already been reported label Apr 3, 2020
@rebron
Copy link
Collaborator

rebron commented Apr 3, 2020

@cezaraugusto Please reopen if you meant something else with this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
closed/duplicate Issue has already been reported feature/new-tab regression
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cezaraugusto @rebron