Skip to content

Improve Emails UX - #28057

Merged
Pytal merged 2 commits into
masterfrom
feat/email-ux
Jul 20, 2021
Merged

Improve Emails UX#28057
Pytal merged 2 commits into
masterfrom
feat/email-ux

Conversation

@Pytal

Copy link
Copy Markdown
Member

Close#28056

  • Autofocus additional inputs when created
  • Allow deleting newly created additional inputs
  • Prevent deletion of empty additional input when hitting backspace

@PytalPytal added enhancement 3. to review Waiting for reviews labels Jul 20, 2021
@PytalPytal added this to the Nextcloud 22.0.1 milestone Jul 20, 2021
@PytalPytal self-assigned this Jul 20, 2021
@Pytal

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@julien-ncjulien-nc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

When Enter key is pressed in an email field, it opens the email scope action menu (local, federated, published). Is it intentional?

@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 20, 2021
@Pytal

Copy link
Copy Markdown
MemberAuthor

When Enter key is pressed in an email field, it opens the email scope action menu (local, federated, published). Is it intentional?

The keyboard control where you may select options with enter and navigate with up and down arrow keys is from the Actions component of nextcloud-vue, so yes it's part of the expected functionality from Actions :)

Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal
Pytal merged commit ff761b8 into masterJul 20, 2021
@Pytal
Pytal deleted the feat/email-ux branch July 20, 2021 19:21
@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable22 failed. Please do this backport manually.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unintuitive UX when saving Emails

4 participants

@Pytal@julien-nc@skjnldsv@juliusknorr