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

Refactor ReleaseNotes with Aphrodite #7172

Merged
merged 1 commit into from
Feb 15, 2017
Merged

Refactor ReleaseNotes with Aphrodite #7172

merged 1 commit into from
Feb 15, 2017

Conversation

luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Feb 10, 2017

Test Plan

  1. Test that updating using BRAVE_UPDATE_VERSION=0.8.3 env variable works correctly
  2. Click "Details"
  3. Check the design of the dialog

Description

Closes #7171

Auditors: @cezaraugusto

  • 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).

Test Plan:

Copy link
Contributor

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apart from comments on original branch (unneeded quotes on object keys) LGTM ++

@luixxiul
Copy link
Contributor Author

@cezaraugusto thanks for reviewing. Based on your feedback, I removed the quotes.

Also I changed h1 to header, which is more meaningful.

Copy link
Contributor

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++!!

Closes #7171

Auditors:

Test Plan:
1. Test that updating using BRAVE_UPDATE_VERSION=0.8.3 env variable works correctly
2. Click "Details"
3. Check the design of the dialog
@@ -106,7 +106,8 @@ const globalStyles = {
aboutPageSectionPadding: '24px',
defaultTabPadding: '0 4px',
defaultIconPadding: '2px',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

defaultIconPadding was rebased on master.

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants