-
Notifications
You must be signed in to change notification settings - Fork 88
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
chore(NcAppSidebar): correctly define and document exposed CSS variables #5861
base: master
Are you sure you want to change the base?
chore(NcAppSidebar): correctly define and document exposed CSS variables #5861
Conversation
Thank you this is a very nice thing :) |
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
01b3b29
to
38a9884
Compare
Force pushed just a commit message typo fix |
@ShGKme you need to update cypress snapshots 🙈 |
I changed nothing 🥲 |
Will do so later today |
Maybe a padding or something else moved like 1px? |
Ok, actually, it is relevant. Variables are defined on an element outside the |
Put it into |
840c422
to
77b2f6f
Compare
☑️ Resolves
--app-sidebar-offset
🖼️ Screenshots
🏁 Checklist
next
requested with a Vue 3 upgrade