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

Small changes of the icons on about pages #6138

Merged
merged 1 commit into from
Dec 11, 2016
Merged

Small changes of the icons on about pages #6138

merged 1 commit into from
Dec 11, 2016

Conversation

luixxiul
Copy link
Contributor

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

On about:bookmarks

  • Aligned fa-file-o favicon on about:bookmarks

On about:preferences#security

  • Restyled widevineInfo (line-height, paddings around the icons, cursor:pointer on them)

Closes #6137

Auditors: @bbondy @bradleyrichter

Test Plan:

  1. Bookmark about:newtab
  2. Open about:bookmarks
  3. Make sure the favicon of the bookmarked new tab has 16px width
  4. Open about:preferences#security
  5. Make sure the icons below "Plugin Settings" are indented
  6. Check the line-height of the subtexts in Google Widevine Support
  7. Check the cursor on the two info icons

screenshot 2016-12-11 23 54 51

screenshot 2016-12-11 23 54 32

On about:bookmarks
- Aligned fa-file-o favicon on about:bookmarks

On about:preferences#security
- Restyled widevineInfo (line-height, paddings around the icons, cursor:pointer on them)

Closes #6137

Auditors: @bbondy @bradleyrichter

Test Plan:
1. Bookmark about:newtab
2. Open about:bookmarks
3. Make sure the favicon of the bookmarked new tab has 16px width
4. Open about:preferences#security
5. Make sure the icons below "Plugin Settings" are indented
6. Check the line-height of the subtexts in Google Widevine Support
7. Check the cursor on the two info icons
@luixxiul luixxiul added the polish Nice to have — usually related to front-end/visual tasks. label Dec 11, 2016
@luixxiul luixxiul added this to the 0.13.0 milestone Dec 11, 2016
@bradleyrichter
Copy link
Contributor

nice polish!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
polish Nice to have — usually related to front-end/visual tasks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants