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

TLD of long domain is not displayed in title tooltip on bravery panel #9089

Closed
luixxiul opened this issue May 27, 2017 · 0 comments
Closed

TLD of long domain is not displayed in title tooltip on bravery panel #9089

luixxiul opened this issue May 27, 2017 · 0 comments
Assignees
@cezaraugusto
Labels
bug feature/shields QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Milestone
0.18.x Release

Comments

@luixxiul
Copy link
Contributor

luixxiul commented May 27, 2017 •
edited
Loading

Test plan

#9491 (comment)

Describe the issue you encountered: TLD of long domain is not displayed in title tooltip on bravery panel.

  • Platform (Win7, 8, 10? macOS? Linux distro?):

  • Brave Version (revision SHA): 0.15.310

  • Steps to reproduce:

    1. Visit https://longextendedsubdomainnamewithoutdashesinordertotestwordwrapping.badssl.com/
    2. Open the bravery panel
    3. Hover on the domain

  • Actual result: only longextendedsubdomainnamewithoutdashesinordertotestwordwrapping is displayed

  • Expected result: longextendedsubdomainnamewithoutdashesinordertotestwordwrapping.badssl.com should be displayed

  • Extra QA steps:
    1.
    2.
    3.

  • Any related issues:
@luixxiul luixxiul added this to the 0.16.200 milestone May 27, 2017
@luixxiul luixxiul added needs-owner ♞ This issue is tagged for an upcoming release but has no owner. needs-triage and removed needs-owner ♞ This issue is tagged for an upcoming release but has no owner. labels May 27, 2017
@cezaraugusto cezaraugusto self-assigned this Jun 13, 2017
cezaraugusto added a commit that referenced this issue Jun 15, 2017
@cezaraugusto
Show full URL for tooltip inside braveryPanel
7e62a68 
Auditors: @luixxiul, @diracdeltas
Close #9089
@cezaraugusto cezaraugusto mentioned this issue Jun 15, 2017
Merged
8 tasks
This was referenced Jul 24, 2017
Closed
Closed
Closed
Closed
@luixxiul luixxiul mentioned this issue Jul 26, 2017
Closed
44 tasks
This was referenced Jul 26, 2017
Closed
Closed
@alexwykoff alexwykoff mentioned this issue Jul 27, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@cezaraugusto cezaraugusto

Labels
bug feature/shields QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Projects
None yet
Milestone
0.18.x Release
Development

No branches or pull requests
4 participants
@luixxiul @cezaraugusto @srirambv @LaurenWags