Skip to content

Replace the 'new collective' form with a modal #504

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.

Sign up for GitHub

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

Merged
mejo- merged 10 commits into from
Feb 21, 2023
Merged

Replace the 'new collective' form with a modal #504

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
0e8a9c1
Move form for a new collective into a Modal
mejo- Jan 10, 2023
cc37c39
Add member picker for new collective modal
mejo- Jan 12, 2023
5a17bef
Add EmptyContent to first step of the dialog
mejo- Feb 14, 2023
47d4e86
Allow to advance to next step by pressing return
mejo- Feb 14, 2023
363b23e
Fix scrolling and cropped bottom in member search results
mejo- Feb 14, 2023
cc45245
Address more design review feedback
mejo- Feb 14, 2023
b3f653f
Adjust Cypress tests to use new dialog for new collective
mejo- Feb 15, 2023
086c996
Cypress: adjust request timeout when leaving a collective
mejo- Feb 15, 2023
7db9554
Address review feedback
mejo- Feb 21, 2023
eca453b
Cypress: Add test for picking members and fix create collective tests
mejo- Feb 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Allow to advance to next step by pressing return 
Signed-off-by: Jonas <[email protected]>
  • Loading branch information
@mejo-
mejo- committed Feb 21, 2023
commit 47d4e860fa62fa226ef1ab4292fd20356efc3273
9 changes: 8 additions & 1 deletion src/components/Nav/NewCollectiveModal.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
:error="nameIsInvalid"
:show-trailing-button="name !== ''"
:label="t('collectives', 'Name of the collective')"
@keypress.enter.prevent="advanceToMembers"
@trailing-button-click="clearName" />
<NcSelect v-else
ref="circleSelector"
Expand Down Expand Up @@ -70,7 +71,7 @@
<NcButton type="primary"
:disabled="!newCollectiveName || nameIsInvalid"
class="modal-buttons-right"
@click="state = 1">
@click="advanceToMembers">
{{ t('collectives', 'Select members') }}
</NcButton>
</div>
Expand Down Expand Up @@ -209,6 +210,12 @@ export default {
this.name = ''
},

advanceToMembers() {
if (this.newCollectiveName && !this.nameIsInvalid) {
this.state = 1
}
},

async getCircles() {
return await this.dispatchGetCircles()
.catch(displayError('Could not fetch circles'))
Expand Down