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

Password change error message UI/UX #3563

Closed
hitenvidhani opened this issue May 2, 2024 · 2 comments
Closed

Password change error message UI/UX #3563

hitenvidhani opened this issue May 2, 2024 · 2 comments
Labels
needs: clarification type: bug Something isn't working work: frontend Related to frontend code in the mathesar_ui directory

Comments

@hitenvidhani
Copy link
Contributor

hitenvidhani commented May 2, 2024

Description

Some issues were discussed in this PR(#3536 (comment)) regarding password change error messages display.

  1. Error messages display in which language when a user in Japanese mode changes password?
  2. Alignment of error message.
  3. Some other issues which I saw while testing this(Video):
    Notice that I have selected Japanese mode but there are many untranslated pieces of text

Expected behavior

  1. Messages should be in Japanese which is happening currently so no changes required there
    image
  2. Alignment of the messages should be in sync with the button above the error message
  3. All the text should be in Japanese, I suppose. If not, I believe at least the info message should be in Japanese.

To Reproduce

Try setting or changing password that it invalid (which doesn't contain any special character)

Environment

  • OS: (eg. macOS 10.14.6; Fedora 32)
  • Browser: (eg. Safari; Firefox)
  • Browser Version: (eg. 13; 73)
  • Other info:

Additional context

Tasks

No tasks being tracked yet.
@hitenvidhani hitenvidhani added needs: triage This issue has not yet been reviewed by a maintainer type: bug Something isn't working labels May 2, 2024
@seancolsen
Copy link
Contributor

@hitenvidhani This issue is a bit hard to understand. Before we proceed, I'd like you to clean it up.

  • You said:

    Messages should be in Japanese which is happening currently so no changes required there

    If you don't think any changes are required, then please remove all reference to this. You've included a screenshot which seems to show things working as you expect. If that's the case, then please remove the screenshot too. It's distracting and confusing as I'm trying to understand the work required here.

  • You said:

    Alignment of the messages should be in sync with the button above the error message

    I'm not sure what you mean specifically. Are you expecting the left edge of the error message to be horizontally aligned with the left edge of the cancel button? If so, I'd rather not spend time worry about that right now. If it's something you feel strongly about, then please open an issued dedicated to discussing this specific problem in isolation. Describe it clearly with a screenshot (of the UI in English).

  • You said:

    Some other issues which I saw while testing this(Video). Notice that I have selected Japanese mode but there are many untranslated pieces of text. All the text should be in Japanese, I suppose. If not, I believe at least the info message should be in Japanese.

    This is good feedback and probably represents some changes we ought to make. But it's really hard to decipher from the way you've described it. Please open a new issue for this specific problem. Clearly describe the steps (sequentially) to reproduce the problem. Describe your expected behavior and your observed behavior. Include a screenshot (which will be easier to follow than your video). Once that ticket is open we can discuss further.

Unless I'm missing something, I think we can close this issue once you've taken the steps above.

@seancolsen seancolsen added needs: clarification work: frontend Related to frontend code in the mathesar_ui directory and removed needs: triage This issue has not yet been reviewed by a maintainer labels May 6, 2024
@hitenvidhani
Copy link
Contributor Author

hitenvidhani commented May 18, 2024

Created a new issue #3603, hence closing this one.

Are you expecting the left edge of the error message to be horizontally aligned with the left edge of the cancel button?

Yeah, but it can be ignored currently as you said.

@hitenvidhani hitenvidhani closed this as not planned Won't fix, can't repro, duplicate, stale May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs: clarification type: bug Something isn't working work: frontend Related to frontend code in the mathesar_ui directory
Projects
None yet
Development

No branches or pull requests

2 participants