-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
fix: UI/UX fixes #7156 #7173
fix: UI/UX fixes #7156 #7173
Conversation
Uffizzi Preview |
packages/nc-gui/components/smartsheet/toolbar/CreateGroupBy.vue
Outdated
Show resolved
Hide resolved
packages/nc-gui/components/smartsheet/toolbar/CreateGroupBy.vue
Outdated
Show resolved
Hide resolved
packages/nc-gui/components/smartsheet/toolbar/CreateGroupBy.vue
Outdated
Show resolved
Hide resolved
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.
- please pull the code once, run lint (
pnpm run lint
) and fix the lint issue(s). - please update the linked issue - tick those items you've resolved in this PR.
if (!modelValue) { | ||
localState.value = dayjs(new Date()) | ||
} |
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.
as discussed with Raju, the behaviour is a bit off. When I open the datetime picker, the current datetime is expected to be selected. However, we only save the value once we click OK
, which means the value shouldn't be saved if I un-focus it. Currently the value will be saved anyway.
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.
Okay, first I tried to set :default-picker-value="dayjs(new Date())"
but this is not woking.
I will check this
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 fixed this.
Now GroupByMenu test is failing that I need to fix as I changed the GroupByMenu UI and now it's similar to SortMenu
Signed-off-by: Raju Udava <86527202+dstala@users.noreply.github.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
…ing the col object Signed-off-by: Pranav C <pranavxc@gmail.com>
…ing the col object Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Change Summary
Change type
rolePermissions
inlib/types.ts
file is imported from wrong file./constants
insted of./acl
Test/ Verification
Provide summary of changes.