Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

EV certificate string overflows #12780

Closed
diracdeltas opened this issue Jan 22, 2018 · 0 comments · Fixed by #12781
Closed

EV certificate string overflows #12780

diracdeltas opened this issue Jan 22, 2018 · 0 comments · Fixed by #12781
Assignees
@cezaraugusto
Labels
polish Nice to have — usually related to front-end/visual tasks. QA/no-qa-needed release-notes/exclude security
Milestone
0.22.x (Release C...

Comments

@diracdeltas
Copy link
Member

follow-up to #11776

actual behavior: if the organization name is very long, it overflows and makes it impossible to see the URL of the page:
screen shot 2018-01-22 at 9 03 22 pm

expected behavior: the EV info box should enforce a max-width such that the URL is always visible
@diracdeltas diracdeltas added security polish Nice to have — usually related to front-end/visual tasks. labels Jan 22, 2018
@diracdeltas diracdeltas added this to the 0.21.x (Developer Channel) milestone Jan 22, 2018
@cezaraugusto cezaraugusto self-assigned this Jan 22, 2018
cezaraugusto added a commit that referenced this issue Jan 22, 2018
@cezaraugusto
enforce max limit to ev cert display
acc9aec 
follow-up of #11776
fix #12780
cezaraugusto added a commit that referenced this issue Jan 22, 2018
@cezaraugusto
enforce max limit to ev cert display
2cac2b4 
follow-up of #11776
fix #12780
cezaraugusto added a commit that referenced this issue Jan 22, 2018
@cezaraugusto
enforce max limit to ev cert display
b98c2fa 
follow-up of #11776
fix #12780
@cezaraugusto cezaraugusto mentioned this issue Jan 22, 2018
Merged
10 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@cezaraugusto cezaraugusto

Labels
polish Nice to have — usually related to front-end/visual tasks. QA/no-qa-needed release-notes/exclude security
Projects
None yet
Milestone
0.22.x (Release Channel)
Development

Successfully merging a pull request may close this issue.

enforce max limit to ev cert display brave/browser-laptop
2 participants
@diracdeltas @cezaraugusto