docs: Merging css-docs > tech-docs #192
Conversation
| import ThemeContainer from '../ThemeContainer.vue'; | ||
| import ThemeSwitcher from '../ThemeSwitcher.vue'; |
There was a problem hiding this comment.
Is ThemeContainer and ThemeSwitcher doing roughly the same thing by any chance?
There was a problem hiding this comment.
As it seems, yes... almost. 😅
...the only big difference is that the ThemeContainer has a slot for content and ThemeSwitcher is only the switch. 😄
But I can look in to if it is possible to combine these in to one. You could just use the ThemeContainer without slotted content to get only the switch. There is also some styling that is different...
| import QrColorTable from '../qr-color-table.vue'; | ||
| import WidthController from '../WidthController.vue'; | ||
| import Box from '../Box.vue'; | ||
| import { IconStarFull32 } from '@warp-ds/icons/vue'; |
There was a problem hiding this comment.
Not a big deal, but maybe we can use just that one icon here instead of pulling in the whole icons/vue package, like so:
import IconStarFull32 from '@warp-ds/icons/vue/star-full-32';
| @@ -126,6 +145,262 @@ export default defineConfig({ | |||
| { text: 'Designers guide', link: '/migration/designers/' }, | |||
| ], | |||
| }, | |||
| { | |||
| text: 'Classes', | |||
There was a problem hiding this comment.
Should this perhaps be named "CSS" or something instead? 🤔
Fix: typo Co-authored-by: Balbina K <[email protected]>
Sets up for merging the content of css-docs into tech-docs.