Skip to content
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

Scroll back down to alias in list after updating existing alias? #2028

Open
elsiehupp opened this issue Sep 5, 2021 · 1 comment
Open

Scroll back down to alias in list after updating existing alias? #2028

elsiehupp opened this issue Sep 5, 2021 · 1 comment

Comments

@elsiehupp
Copy link
Contributor

elsiehupp commented Sep 5, 2021

Right now, when you click the pencil icon next to an alias in the "Existing Email Aliases" list under "Mail & Users > Aliases", the page scrolls you up to the form at the top of the list to edit the alias.

It is not possible to edit an alias by entering it in the form at the top of the page; if you try to do this you will get a popover saying "Alias already exists <alias>." It would probably be preferable if there were a way to edit an existing alias from the top of the page—maybe by adding an "Edit..." button to the popover—but that's a separate issue.

After you change the "Forwards To" field an click "Update", you will get a popover saying alias updated, and then... nothing. The page does not scroll down and return you to where you started. If you want to continue from before, you need to scroll down yourself (or use in-page search).

Needless to say, this process makes a workflow involving changing multiple aliases more cumbersome than it needs to be. In the title for this Issue, I suggest automatically scrolling back to the place where the user started, but the more ideal solution would be having an in-place update form. The reason to solve the problem by automatically scrolling back down would be if scrolling to an anchor is easier to implement than the in-place form.


Oh, and somewhat oddly, the popovers for "Alias already exists <alias>." and alias updated use different styling, with the former including the alias in question and the latter not including the alias and additionally using monospace characters. This isn't in any way a breaking issue, but it seems like the popovers could both be changed to improve readability, to "Alias already exists." and "Alias <alias> updated.", with the alias in question possibly in monospace characters.

This is obviously also a separate issue, but I could look into it myself to begin, and I imagine it being more within my ability to fix than the others.

Alias already exists

alias updated

@elsiehupp
Copy link
Contributor Author

Update: I've found some potential fixes, but I need to finish setting up a test environment on my computer before I can open one or more pull requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant