-
Notifications
You must be signed in to change notification settings - Fork 20
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
Edit-menu: Modify interface from users feedback #1371
Edit-menu: Modify interface from users feedback #1371
Conversation
src/components/EditMenu.vue
Outdated
// Resize mini widgets so they fit the layout when the widget mode is set to mini widgets | ||
const miniWidgetContainers = ref<Record<string, HTMLElement>>({}) | ||
watch(widgetMode, () => { | ||
if (widgetMode.value !== 'Mini widgets') return | ||
nextTick(() => { | ||
Object.values(miniWidgetContainers.value).forEach((element) => { | ||
if (element.scrollWidth > element.clientWidth) { | ||
let scale = 1 | ||
while (element.scrollWidth > element.clientWidth) { | ||
scale -= 0.01 | ||
const actualElement = element.children[1] as HTMLElement | ||
actualElement.style.scale = `${scale}` | ||
} | ||
} | ||
}) | ||
}) | ||
}) |
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.
I think you accidentally removed a feature here.
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.
Oops, Thanks for letting me know
4731d35
to
a405dcd
Compare
There's just one thing that I remember @ES-Alexander mentioning that was not added here: the ability to click outside of the profile-select-menu and it closing. Can we also add that here? |
Done, @rafaellehmkuhl |
a405dcd
to
0d47248
Compare
Signed-off-by: Arturo Manzoli <[email protected]>
…-mode Signed-off-by: Arturo Manzoli <[email protected]>
0d47248
to
c3afe0e
Compare
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.
@ArturoManzoli everything seems fine here. Can I merge?
Yep, can merge! |
@ArturoManzoli I was using edit-mode and noticed the problem came back: Can you take a look? |
From @ES-Alexander feedback: