-
Notifications
You must be signed in to change notification settings - Fork 30
Conversation
Nothing in this doc should be considered final until it is merged!
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.
Mostly some typos and minor suggestions, otherwise looks good 🎉
I am very excited about more themes, and in the future custom themes!
configuration/themes.md
Outdated
|
||
### Known issues | ||
We will not have formal UI and syntax themes the like Atom, but will provide a way to override syntax styles (and more) from a single theme. |
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.
We will not have formal UI and syntax themes the like Atom, but will provide a way to override syntax styles (and more) from a single theme. | |
We will not have formal UI and syntax themes like Atom, but will provide a way to override syntax styles (and more) from a single theme. |
I think the
is not needed here?
Co-authored-by: Robert Clover <[email protected]>
Co-authored-by: Robert Clover <[email protected]>
Co-authored-by: Sebastian L. K. Sørensen <[email protected]>
Co-authored-by: Sebastian L. K. Sørensen <[email protected]>
Thanks for the comments/corrections y'all. Going to hack on this a bit on the plane. This may still be open for a while while we develop the vision for what will be in the initial themes 1.0 spec. |
Co-authored-by: Sebastian L. K. Sørensen <[email protected]>
Co-authored-by: Sebastian L. K. Sørensen <[email protected]>
Co-authored-by: Robert Clover <[email protected]>
Co-authored-by: Sebastian L. K. Sørensen <[email protected]>
Co-authored-by: pr <[email protected]>
Co-authored-by: pr <[email protected]>
Co-authored-by: pr <[email protected]>
Co-authored-by: pr <[email protected]>
Thanks all for catching all my typos ❤️ |
Pushed a few tiny changes and then fixed the conflict with main. Things look fine to me - but you certainly know the status of theme stuff more than I do 🤣 |
Awesome gunna merge it! Thanks for contributing everyone. |
This PR updates the theme docs with more of our vision and roadmap for themes. It also removes some outdated theme details.