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

add style to live mode #1092

Merged

Conversation

tomivm
Copy link
Collaborator

@tomivm tomivm commented Dec 8, 2021

close #1071 close #1051

Copy link
Collaborator

@martinbedouret martinbedouret left a comment

Choose a reason for hiding this comment

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

awesome job, congrats @tomivm

src/components/Board/Symbol/Symbol.css Outdated Show resolved Hide resolved
@martinbedouret martinbedouret added this to the 1.17.0 milestone Dec 10, 2021
@tomivm tomivm changed the title Feature/add style to live mode add style to live mode Dec 15, 2021
@tomivm tomivm force-pushed the feature/add-style-to-live-mode branch from 4053e93 to 9c89e45 Compare December 15, 2021 16:21
@tomivm
Copy link
Collaborator Author

tomivm commented Dec 15, 2021

Thanks for the review @martinbedouret . I made some changes to fix issues if 'Remove symbols from the output bar' is enabled. There is a test failing because I use CSS vars. I can't find a solution to disable that warning. It' looks like Csslint is outdated. you can see this same issue. Can you ignore the issue on Codacy?
The browser support to CSS vars is really high. Only IE does not support it.

@martinbedouret martinbedouret merged commit 9af25e9 into cboard-org:master Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Live mode labels not visible in dark mode text on Live feature is not properly aligned.
2 participants