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

Add compact bravery panel with Aphrodite #8954

Closed
luixxiul opened this issue May 19, 2017 · 6 comments
Closed

Add compact bravery panel with Aphrodite #8954

luixxiul opened this issue May 19, 2017 · 6 comments
Assignees
@luixxiul
Labels
feature/shields l10n QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Milestone
0.17.13 (Release ...

Comments

@luixxiul
Copy link
Contributor

luixxiul commented May 19, 2017 •
edited
Loading

Test plan

#8990 (comment)

Describe the issue you encountered: Add compact bravery panel with Aphrodite. The panel should be l10n-friendly as well.

The panel was updated with #9077

screenshot 2017-05-19 17 55 47

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:
@luixxiul luixxiul self-assigned this May 19, 2017
@luixxiul luixxiul added the l10n label May 19, 2017
@NejcZdovc
Copy link
Contributor

What happens when domain is really long?

@luixxiul
Copy link
Contributor Author

long

@NejcZdovc
Copy link
Contributor

I like it 👍

@luixxiul luixxiul added this to the 0.15.400 milestone May 19, 2017
@luixxiul luixxiul mentioned this issue May 21, 2017
Merged
7 tasks
This was referenced May 24, 2017
Closed
Merged
@luixxiul luixxiul mentioned this issue Jun 4, 2017
Merged
7 tasks
This was referenced Jun 19, 2017
Closed
Closed
Closed
Closed
Closed
@Jacalz Jacalz mentioned this issue Jun 21, 2017
Closed
94 tasks
@alexwykoff alexwykoff mentioned this issue Jun 22, 2017
Closed
@LaurenWags LaurenWags mentioned this issue Jun 23, 2017
Closed
@bsclifton
Copy link
Member

bsclifton commented Jun 26, 2017 •
edited
Loading

I'm running the latest preview and it's not going compact for me (even when I resize the window to be small)

Q: Is there anything special that needs to be done?

@bsclifton
Copy link
Member

bsclifton commented Jun 26, 2017 •
edited
Loading

A: after reading the test plan, yes- gotta enable the option 😛

This is awesome <3

@luixxiul
Copy link
Contributor Author

yes, it's disabled by default.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@luixxiul luixxiul

Labels
feature/shields l10n QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Projects
None yet
Milestone
0.17.13 (Release Channel)
Development

No branches or pull requests
5 participants
@luixxiul @bsclifton @NejcZdovc @srirambv @LaurenWags