refactor(appstore): migrate to Typescript and Vue 3#57290
Conversation
3a93b94 to
d0810b4
Compare
d0810b4 to
3f922df
Compare
4843981 to
3ac9531
Compare
047e3f1 to
e9a6556
Compare
e9a6556 to
36a37f5
Compare
62187b7 to
c553e4a
Compare
36c3be6 to
17132cc
Compare
artonge
left a comment
There was a problem hiding this comment.
Good got me besides some nitpicks
| } | ||
| } | ||
|
|
||
| .officeSuiteSwitcherItem_selected { |
There was a problem hiding this comment.
Also, any reason to not using scss to nest the blocks?
There was a problem hiding this comment.
Its BEM:
- Block:
officeSuiteSwitcherItem - Element: none (its the root)
- Modifier:
selected
Its just using _ instead of - (as per BEM naming guide) as otherwise you cannot properly use it with dot notation like
$style.officeSuiteSwitcherItem_selected vs $style['officeSuiteSwitcherItem--selected']
There was a problem hiding this comment.
Also, any reason to not using scss to nest the blocks?
No need anymore for SCSS, nowadays this is supported by native CSS nesting.
I also saw many complaining about deep nesting and here no nesting is needed for CSS rule specificity.
Or do you mean for things like this:
.base {
&__sub {
// ...
}
}There was a problem hiding this comment.
Its just using
_instead of-(as per BEM naming guide) as otherwise you cannot properly use it with dot notation like
$style.officeSuiteSwitcherItem_selectedvs$style['officeSuiteSwitcherItem--selected']
I have no preference, but if this is nicer to use in practice, then we should update our recommendations.
Or do you mean for things like this:
.base { &__sub { // ... } }
Yes, I was thinking of that. And similarity, if this leads to complaints due to some reason, maybe we should reconsider our recommendations.
| <col :style="{ width: isNarrow ? '60%' : '45%' }"> | ||
| <col> | ||
| <col v-if="!isNarrow" style="width: 15%"> | ||
| <col :style="{ width: isNarrow ? 'calc(3 * var(--default-grid-baseline) + 2 * var(--default-clickable-area))' : '25%' }"> |
There was a problem hiding this comment.
IIRC, css supports block specific break points so that this can be done 100% in css, but that might not be broadly available yet.
There was a problem hiding this comment.
As far as I know there are no element specific break points.
Only viewport based breakpoints.
But moved it to the styles
There was a problem hiding this comment.
Or am I misunderstanding?
https://modern-css.com/responsive-components-without-media-queries/
15af1c0 to
b92937e
Compare
|
/compile rebase |
reduces transfered data from 6MiB to <1MiB Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
… Vue 3 Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
b92937e to
7d8e640
Compare
Limit to groupsselection broken for >~5 groups #39322Summary
settings+appstoreto separate the meansappstoreapp to register root URLs (to prevent breaking changes in the URLs)Checklist
3. to review, feature component)stable32)