Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow styling of radio button backgrounds #5791

Merged
merged 1 commit into from
Apr 22, 2017

Conversation

megawac
Copy link
Contributor

@megawac megawac commented Apr 17, 2017

This fixes radio buttons to use @btn-default-bg as the background by default instead of @component-bg. This was causing issues with our theme as radio buttons were using @btn-default-color which was not compatible with our default @component-bg.

This also adds 2 variables which correspond to the @btn-default-* settings by default

  • @btn-radio-bg
  • @btn-radio-color

@mention-bot
Copy link

@megawac, thanks for your PR! By analyzing the history of the files in this pull request, we identified @afc163, @jljsj33 and @simaQ to be potential reviewers.

@megawac
Copy link
Contributor Author

megawac commented Apr 17, 2017

fyi, build failure isn't from this change. Master is currently failing (https://travis-ci.org/ant-design/ant-design/builds/222804823)

@benjycui
Copy link
Contributor

Rebase, please.

@codecov
Copy link

codecov bot commented Apr 19, 2017

Codecov Report

Merging #5791 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5791      +/-   ##
==========================================
- Coverage   86.83%   86.81%   -0.03%     
==========================================
  Files         183      183              
  Lines        4557     4557              
  Branches     1320     1318       -2     
==========================================
- Hits         3957     3956       -1     
- Misses        600      601       +1
Impacted Files Coverage Δ
components/date-picker/createPicker.tsx 84.31% <0%> (-1.69%) ⬇️
components/date-picker/wrapPicker.tsx 86.2% <0%> (-0.46%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5fe3cb9...682b998. Read the comment docs.

@megawac
Copy link
Contributor Author

megawac commented Apr 19, 2017

done

@benjycui benjycui requested a review from afc163 April 20, 2017 01:28
@@ -130,6 +130,10 @@

@btn-group-border : @primary-7;

// Radio buttons
@btn-radio-bg : @btn-default-bg;
@btn-radio-color : @btn-default-color;
Copy link
Member

Choose a reason for hiding this comment

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

radio-button-bg radio-button-color

@megawac
Copy link
Contributor Author

megawac commented Apr 20, 2017

Done

@ddcat1115 ddcat1115 merged commit 1790642 into ant-design:master Apr 22, 2017
@lock
Copy link

lock bot commented May 1, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators May 1, 2018
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.

None yet

5 participants