-
Notifications
You must be signed in to change notification settings - Fork 4.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
Replace auto-complete link mini dlg with slider #13163
Replace auto-complete link mini dlg with slider #13163
Conversation
<Item id="6860" name="Matched pair 1:"/> | ||
<Item id="6863" name="Matched pair 2:"/> | ||
<Item id="6866" name="Matched pair 3:"/> | ||
<Item id="6860" name="Matched pair 1 :"/> |
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.
Why we add space before ":"? Is it because of the visual improvement? I've checked several programs and none add spaces here. We currently have an open issue regarding this and I don't know what to do with it.
#4628
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 did not know about that issue. I just wanted to keep it consistent.
IMHO I would not add space before :
.
Very nice! 👍 Thank you. |
@Yaron10 |
Great. Thank you for your work. |
1f53672
to
ea9c618
Compare
ea9c618
to
1c45f50
Compare
May I know which ones? |
fix #13162