Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Small fixes #806

Merged
merged 6 commits into from
Sep 8, 2017
Merged

Small fixes #806

merged 6 commits into from
Sep 8, 2017

Conversation

jjeffryes
Copy link
Contributor

This PR addresses a variety of small visual issues.

  • the beta label has a tooltip if the user is not in testnet mode.
  • the superfluous info icon next to the coupon label in purchase was removed
  • a placeholder was added for the coupon input in purchase
  • the coupon input being blank is now handled separately from an invalid code
  • the ... button on the user page was removed
  • the windows controls were updated to look more like Mac controls and be centered properly
  • other nav bar icons that were off center vertically were fixed

Closes #789, 790, 791, 788, and 786

@rmisio
Copy link
Contributor

rmisio commented Sep 7, 2017

The middle icon gets an underline when you hover over it. I'm assuming it shouldn't since the others don't and it arguably looks cleaner without.

image

@morebrownies
Copy link

Noticed the same issue that @rmisio reported above too. Otherwise, looks good to me.

@jjeffryes
Copy link
Contributor Author

Huh. Good catch. I'm not sure how that happened. Will investigate and update later tonight.

@jjeffryes
Copy link
Contributor Author

@rmisio underline removed.

@jjeffryes jjeffryes merged commit 29492fc into master Sep 8, 2017
@jjeffryes jjeffryes deleted the smallFixes branch September 8, 2017 03:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'Beta' Shows as a Button and Does Nothing
3 participants