Skip to content
This repository has been archived by the owner. It is now read-only.

Replace float:right with flex model on alert dialogs #7674

Merged
merged 1 commit into from Mar 13, 2017

Conversation

@luixxiul
Copy link
Contributor

luixxiul commented Mar 13, 2017

Test Plan:

  1. Visit http://jsbin.com/fiyojusahu/edit?html,output
  2. In the output area, click the Click me to test an alert button
  3. Make sure the buttons are aligned to the right
  4. Visit http://jsbin.com/sadunogefu/edit?html,output
  5. In the output area, click the Click me to test a confirm button
  6. Make sure the buttons are aligned to the right

Description

Follow-up of #7107 (#7107 (comment))

Closes #7673

Auditors: @bsclifton, @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).
Suguru Hirahara
Follow-up of #7107 (#7107 (comment))

Closes #7673

Auditors:

Test Plan:
1. Visit http://jsbin.com/fiyojusahu/edit?html,output
2. In the `output` area, click the `Click me to test an alert` button
3. Make sure the buttons are aligned to the right
4. Visit http://jsbin.com/sadunogefu/edit?html,output
5. In the `output` area, click the `Click me to test a confirm` button
6. Make sure the buttons are aligned to the right
@luixxiul luixxiul added this to the 0.13.6 milestone Mar 13, 2017
@luixxiul luixxiul self-assigned this Mar 13, 2017
@luixxiul luixxiul requested review from cezaraugusto and bsclifton Mar 13, 2017
Copy link
Member

bsclifton left a comment

Manually tested on Windows 10, looks great! 😄

@bsclifton bsclifton merged commit 3378e7f into brave:master Mar 13, 2017
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Copy link
Contributor

cezaraugusto left a comment

++

@luixxiul luixxiul deleted the luixxiul:alert-button-flexmodel branch Mar 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.