-
-
Notifications
You must be signed in to change notification settings - Fork 47.5k
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
Add a less variable to opt out of form item colons #5855
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5855 +/- ##
=======================================
Coverage 86.82% 86.82%
=======================================
Files 183 183
Lines 4560 4560
Branches 1323 1323
=======================================
Hits 3959 3959
Misses 601 601 Continue to review full report at Codecov.
|
components/style/themes/default.less
Outdated
@@ -194,6 +194,7 @@ | |||
@label-required-color : @highlight-color; | |||
@label-color : @text-color; | |||
@form-item-margin-bottom : 24px; | |||
@form-item-trailing-colon : false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should default to true
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops thanks, didnt mean to commit it as false
c35443c
to
297412b
Compare
I also remove colons |
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. |
Colons disabled (@form-item-trailing-colon: false):
Colons enabled (@form-item-trailing-colon: true):
Our designer wanted to opt out of manadatory colons here