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

Improve typography in Brave Payments notification panel #12770

Closed
bradleyrichter opened this issue Jan 22, 2018 · 16 comments
Closed

Improve typography in Brave Payments notification panel #12770

bradleyrichter opened this issue Jan 22, 2018 · 16 comments
Assignees
Labels
design A design change, especially one which needs input from the design team. feature/rewards priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). QA/test-plan-specified release-notes/include
Milestone

Comments

@bradleyrichter
Copy link
Contributor

bradleyrichter commented Jan 22, 2018

Test plan

  1. Launch brave with LEDGER_ENVIRONMENT=staging
  2. Visit about:preferences, go to payments tab
  3. Enable payments; claim tokens
  4. Alignment for text should look like this:
    image

Original issue description

Sub-text issues to improve:

  • line spacing too tight
  • text color too light
  • string width to narrow, creating orphans in EN strings

Current:
image

Improvements:

  • increase line spacing by 30%
  • darken grey value by 30%
  • extend text area width by 50px

image

@bradleyrichter bradleyrichter added design A design change, especially one which needs input from the design team. feature/rewards labels Jan 22, 2018
@bradleyrichter bradleyrichter added this to the 0.21.x (Developer Channel) milestone Jan 22, 2018
@arsalankhalid
Copy link
Contributor

Hey guys, is this still available - would love to take a stab at this :)

@bradleyrichter
Copy link
Contributor Author

@arsalankhalid We'd love your help! Go for it...

cc @NejcZdovc

@NejcZdovc
Copy link
Contributor

@arsalankhalid yes it's still available. If you need any help let me know. Thank you

@arsalankhalid
Copy link
Contributor

Great, thanks for passing it over - will reach out :)

@arsalankhalid
Copy link
Contributor

@NejcZdovc hard to re-create this, what are your thoughts?

image

Is this fixed/non-existent, or an edge case based on specific messages the user received, just as the million dollar give away example, how can I re-create that?

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Feb 4, 2018

@arsalankhalid try starting your run with LEDGER_ENVIRONMENT=staging npm run start (don't forget to clean profile). We still have some promotions on our staging servers. Please, while you are working on this one, just claim one promotion and then don't click OK or X until you are done. This way you will not run out of promotions.

@arsalankhalid
Copy link
Contributor

Great thanks, let me give that a shot.

@arsalankhalid
Copy link
Contributor

Sorry for the radio silence, been a bit busy. Will take a stab tomorrow I more than hope :)

@arsalankhalid
Copy link
Contributor

Hey still not sure how to re-create, how exactly do i 'claim' a promo?

@NejcZdovc
Copy link
Contributor

@NejcZdovc NejcZdovc modified the milestones: 0.21.x (Beta Channel), 0.22.x (Developer Channel) Feb 14, 2018
@arsalankhalid
Copy link
Contributor

Sadly don't see it when I run it, even in staging :(

image

I was hoping there was a way to test running the promos, or not possible? It would be easier to ensure the typography is good for all types of promo messages...as long as those promos are actually fake :P

It's also possible i tried to run the promo already, in staging - but not sure. Please advise

@NejcZdovc
Copy link
Contributor

Archive.zip

You can use this profile. Just copy it into user profile folder and run LEDGER_ENVIRONMENT=stagingn npm run start and you should see the overlay that you need.

@arsalankhalid
Copy link
Contributor

I hate to be the local noob, but wheres our user profile? Trying to find Brave directory on OSX. Almost seems like a feature for users to easily switch their profiles through preferences/settings? I saw some open and closed tasks regarding to 'portable profiles', but don't see anything in preferences to easily switch.

Also that archive.zip file looks sketchy at first look lol, but i guess its a profile :P

@NejcZdovc
Copy link
Contributor

your user profile is in /Users/username/Library/Application Support. In there you will find brave-development folder. Delete whatever is in this folder and extract zip into it.

@arsalankhalid
Copy link
Contributor

in my brave-development folder, but don't see anything:

image

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Feb 15, 2018

@arsalankhalid if you have time we can jump on a call and check it out. Just send me an email to nejc[at]brave.com and will send you link to the zoom room

@bbondy bbondy modified the milestones: 0.22.x (Developer Channel), 0.23.x (Nightly Channel) Feb 25, 2018
arsalankhalid pushed a commit to arsalankhalid/browser-laptop that referenced this issue Mar 4, 2018
@alexwykoff alexwykoff added the priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). label Mar 13, 2018
@alexwykoff alexwykoff modified the milestones: 0.23.x (Nightly Channel), Completed work Mar 13, 2018
@alexwykoff alexwykoff removed this from the Completed work milestone Mar 20, 2018
@alexwykoff alexwykoff added this to the 0.24.x (Nightly Channel) milestone Mar 20, 2018
bsclifton added a commit that referenced this issue Mar 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
design A design change, especially one which needs input from the design team. feature/rewards priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). QA/test-plan-specified release-notes/include
Projects
None yet
Development

No branches or pull requests

6 participants