-
Notifications
You must be signed in to change notification settings - Fork 42
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
17152 Menu UI improvements #635
17152 Menu UI improvements #635
Conversation
- now return item object so we don't have to find it - added class to disabled items - now include but disable items not in active group - hide disabled list items via CSS
@@ -17,8 +17,15 @@ | |||
item-value="[group,value]" | |||
:menu-props="{ bottom: true, offsetY: true, maxHeight: 423 }" | |||
@change="setClearErrors(null); onRequestActionChange($event)" | |||
return-object |
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.
This returns the entire item object (in $event
) instead of just its value.
<!-- <template #selection="{ item }"> | ||
<div class="font-weight-bold text-truncate">{{ item.text }}</div> | ||
<div class="text-subtitle-1">{{ item.subtext }}</div> | ||
</template> --> |
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.
/gcbrun |
@@ -32,9 +39,12 @@ | |||
</div> | |||
</v-list-item-content> | |||
|
|||
<!-- render but conditionally hide disabled list items, so that the v-select | |||
continues to display the current selection even when a different group is active --> |
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.
If the item is not in the list then the v-select sometimes shows a blank selection (ticket 17044).
<v-list-item-content | ||
v-else | ||
class="group-item pl-8 pr-4 py-4" | ||
:class="{ 'hide-me': item.disabled }" |
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.
Can't just use d-none
because it's actually the parent element I need to hide.
Temporary Url for review: https://namerequest-dev--pr-635-hnknd6zs.web.app |
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.
Looks great Sev! 👍
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.
LGTM
ef3d33c
into
bcgov:feature-way-of-navigating
- now return item object so we don't have to find it - added class to disabled items - now include but disable items not in active group - hide disabled list items via CSS
- now return item object so we don't have to find it - added class to disabled items - now include but disable items not in active group - hide disabled list items via CSS
- now return item object so we don't have to find it - added class to disabled items - now include but disable items not in active group - hide disabled list items via CSS
- now return item object so we don't have to find it - added class to disabled items - now include but disable items not in active group - hide disabled list items via CSS
Issue #: /bcgov/entity#17152
Description of changes:
- app version = 5.0.12
- now return item object so we don't have to find it
- added class to disabled items
- now include but disable items not in active group
- hide disabled list items via CSS
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namerequest license (Apache 2.0).