-
-
Notifications
You must be signed in to change notification settings - Fork 681
Improved rule list doc #1860
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
Improved rule list doc #1860
Conversation
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.
Really cool changes!
Maybe the layout of only this docs page can be adjusted, so it is wider (on screens with more horizontal space) and thus better uses the available space?
Co-authored-by: Flo Edelmann <[email protected]>
Co-authored-by: Flo Edelmann <[email protected]>
Co-authored-by: Flo Edelmann <[email protected]>
Co-authored-by: Flo Edelmann <[email protected]>
Co-authored-by: Flo Edelmann <[email protected]>
Co-authored-by: Flo Edelmann <[email protected]>
Co-authored-by: Flo Edelmann <[email protected]>
Co-authored-by: Flo Edelmann <[email protected]>
I think I made the changes you requested. Could you check it again? |
</template> | ||
|
||
<script> | ||
const emojis = ['3️⃣', '2️⃣', '⚠️'] |
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.
Intl.Segmenter
is required to properly split these emojis, but FireFox doesn't support it, so we'll handle it ourselves.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/Segmenter
This PR improves the rule list documentation.
There are quite a lot of rules now. I have made the following changes to make the rules easier for users to access.
https://deploy-preview-1860--eslint-plugin-vue.netlify.app/rules/