This repository has been archived by the owner on Jan 15, 2021. It is now read-only.
[V2 button] Rearrange component file structure and small prop fixes #1640
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* _type > variation * _size > size
Velenir
approved these changes
Nov 23, 2020
anxolin
approved these changes
Nov 23, 2020
W3stside
added a commit
that referenced
this pull request
Jan 5, 2021
* added styled-theming lib * styling sheets 1. colours 2. styles 3. commented out stuff in theme * simplify ButtonBase code + add stories * adjust darkMode decorator for new theme 1, add colour to colourSheet * StyledButton base * addressed PR comments: 1. added type to Story 2. Changed DarkThemeTOggler to ThemeToggler 3. simplified ButtonVariant enum * fix button type, add disabled button, string enum 1. use string enum as button type 2. default to dark mode on start * [V2] Colour Sheet from Invision wireframes + added to Button + stories (#1632) * cleanup button story * new colour sheet * fix type, remove class styles, use const style * use new colours in StyledButton * misc. 1. add borderRadius to const 2. use theme bg for darkMode switch * [V2] Button sizes (#1633) * wrap our base styled button in own theme provider and add size opt * add proper types for button size * Stories: add B I G and smol button stories * misc. 1. make theme toggle button smol 2. add buttonFontSize to styles sheet * enum > type for ButtonVariations * incorrect darkMode button logo * better prop names (#1637) 1. kind > $type 2. size > $size * [V2 button] Rearrange component file structure and small prop fixes (#1640) * saving * $transient name props not allowed - use _propName * change props again * _type > variation * _size > size * variation > variant prop name
W3stside
added a commit
that referenced
this pull request
Jan 7, 2021
* creating theme pt 1 * theme * DarkModeThemeToggler decorator * Button common component * Button story * cleaned up Theme types * simplify ButtonBase prop logic * unmemo fn * fix story bprops * V2/button theming (#1626) * added styled-theming lib * styling sheets 1. colours 2. styles 3. commented out stuff in theme * simplify ButtonBase code + add stories * adjust darkMode decorator for new theme 1, add colour to colourSheet * StyledButton base * addressed PR comments: 1. added type to Story 2. Changed DarkThemeTOggler to ThemeToggler 3. simplified ButtonVariant enum * fix button type, add disabled button, string enum 1. use string enum as button type 2. default to dark mode on start * [V2] Colour Sheet from Invision wireframes + added to Button + stories (#1632) * cleanup button story * new colour sheet * fix type, remove class styles, use const style * use new colours in StyledButton * misc. 1. add borderRadius to const 2. use theme bg for darkMode switch * [V2] Button sizes (#1633) * wrap our base styled button in own theme provider and add size opt * add proper types for button size * Stories: add B I G and smol button stories * misc. 1. make theme toggle button smol 2. add buttonFontSize to styles sheet * enum > type for ButtonVariations * incorrect darkMode button logo * better prop names (#1637) 1. kind > $type 2. size > $size * [V2 button] Rearrange component file structure and small prop fixes (#1640) * saving * $transient name props not allowed - use _propName * change props again * _type > variation * _size > size * variation > variant prop name
W3stside
added a commit
that referenced
this pull request
Jan 7, 2021
* creating theme pt 1 * theme * DarkModeThemeToggler decorator * Button common component * Button story * cleaned up Theme types * simplify ButtonBase prop logic * unmemo fn * fix story bprops * V2/button theming (#1626) * added styled-theming lib * styling sheets 1. colours 2. styles 3. commented out stuff in theme * simplify ButtonBase code + add stories * adjust darkMode decorator for new theme 1, add colour to colourSheet * StyledButton base * addressed PR comments: 1. added type to Story 2. Changed DarkThemeTOggler to ThemeToggler 3. simplified ButtonVariant enum * fix button type, add disabled button, string enum 1. use string enum as button type 2. default to dark mode on start * [V2] Colour Sheet from Invision wireframes + added to Button + stories (#1632) * cleanup button story * new colour sheet * fix type, remove class styles, use const style * use new colours in StyledButton * misc. 1. add borderRadius to const 2. use theme bg for darkMode switch * [V2] Button sizes (#1633) * wrap our base styled button in own theme provider and add size opt * add proper types for button size * Stories: add B I G and smol button stories * misc. 1. make theme toggle button smol 2. add buttonFontSize to styles sheet * enum > type for ButtonVariations * incorrect darkMode button logo * better prop names (#1637) 1. kind > $type 2. size > $size * [V2 button] Rearrange component file structure and small prop fixes (#1640) * saving * $transient name props not allowed - use _propName * change props again * _type > variation * _size > size * variation > variant prop name * add polished, css interpolation pkg
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes