-
Notifications
You must be signed in to change notification settings - Fork 173
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
feat(fe2): invite + list workspace invites #2629
Conversation
show-clear | ||
:placeholder="searchPlaceholder" | ||
class="rounded-md border border-outline-3" | ||
v-bind="bind" |
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.
Improved useDebouncedTextInput
return to allow for way simpler/easier configuration of the FormTextInput component
@@ -92,7 +92,7 @@ | |||
</div> | |||
<div | |||
v-if="hasButtons" | |||
class="relative z-50 flex px-6 py-3 space-x-3 shrink-0 bg-foundation-page border-t border-outline-2" | |||
class="relative z-50 flex px-6 py-3 space-x-3 shrink-0 bg-foundation-page" |
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.
None of Michal's new dialogs have a border above the button footer
@@ -21,7 +20,17 @@ | |||
class="divide-y divide-outline-3 h-full overflow-visible" | |||
:class="{ 'pb-32': overflowCells }" | |||
> | |||
<template v-if="items.length"> | |||
<div | |||
v-if="loading || !items" |
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.
Undefined items
now means - show loading bar (for easy feeding in of API data). Additionally you can manually enable the loading bar through loading
@@ -0,0 +1,157 @@ | |||
<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.
Moved+Updated Mike's table
@@ -313,23 +313,23 @@ const revolveFieldNameAndVariables = < | |||
* full objects. Read more: https://www.apollographql.com/docs/react/caching/cache-interaction/#values-vs-references | |||
*/ | |||
export function modifyObjectFields< | |||
V extends Optional<Record<string, unknown>> = undefined, | |||
D = unknown | |||
Variables extends Optional<Record<string, unknown>> = undefined, |
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.
<3
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.
Nice 😁 Just one small thingy
No description provided.