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

refactor: unify styling #296

Merged

Conversation

magicmatatjahu
Copy link
Member

@magicmatatjahu magicmatatjahu commented Apr 8, 2021

Description

Changes proposed in this pull request:

Unify styling in component with HTML-Templates styling:

  • styling Info, Server, Operation, Message, Schema components
  • the number of lines is related with new dependencies like postcss, tailwind etc. The rest of the code is converting inline styles/classes into mixed classes.

In next PR will be styling for Examples, Sidebar and for mobiles.

Examples:

  • Info:
    image

  • Server
    image

  • Operation
    image

  • Message
    image

Related issue(s)
Part of asyncapi/shape-up-process#86

@magicmatatjahu magicmatatjahu added enhancement New feature or request area/library Related to all activities around Library package labels Apr 8, 2021
@magicmatatjahu magicmatatjahu marked this pull request as ready for review April 9, 2021 11:21
library/src/styles/styles.css Outdated Show resolved Hide resolved
@magicmatatjahu
Copy link
Member Author

@fmvilas I moved the styles to components files :) Could you check again? I also moved those that are repeated, such as for buttons, because you are right that it is better to think about theming in the next cycles.

@sonarcloud
Copy link

sonarcloud bot commented Apr 14, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Member

@fmvilas fmvilas left a comment

Choose a reason for hiding this comment

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

Looking great 👍 Thanks for taking my opinion into consideration! 🙏

@magicmatatjahu magicmatatjahu merged commit 477f015 into asyncapi:unify-component Apr 15, 2021
@magicmatatjahu magicmatatjahu deleted the unify-styling-1 branch April 15, 2021 08:14
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 0.21.0-next.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/library Related to all activities around Library package enhancement New feature or request released on @next
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants