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

Org Model: Update "view only" message on the domains page and fix tooltip #2775

Closed
2 tasks done
Katherine-Osos opened this issue Sep 18, 2024 · 5 comments
Closed
2 tasks done
Assignees
Labels
bug Something that isn't working as intended carryover Carryover from a previous sprint dev Feature: 🏢 Org Model

Comments

@Katherine-Osos
Copy link
Contributor

Katherine-Osos commented Sep 18, 2024

Issue description

Domain page: We need to update the message that displays if a user can only view a domain, but not manage it.

Domains table: We also need to fix the tooltip that appears when hovering over the "On Hold" status. The tooltip gets cut off when there are multiple domains in the table with different statuses. Note: the tooltip doesn't seem to get cut off if the "On Hold" domain is the only one listed in the table.

Acceptance criteria

  • On the domain page, update the message that appears if a user can only view a domain, but not manage it. Use the text in this document.
  • In the domain table, fix the tooltip that appears when hovering over a domain in "on hold" status. The beginning of the tooltip gets cut off.

Additional context

Beginning of tooltip gets cut off:

Image

Links to other issues

No response

@rachidatecs
Copy link
Contributor

@Katherine-Osos @abroddrick I just reviewed this ticket. The tooltip getting cut off is due to how the responsive table wrapper sets the overflow. This could prove to be really tricky to solve. A easier approach would be to shorten the tooltip text (could work, would need testing). Is that an option?

@CocoByte CocoByte self-assigned this Oct 1, 2024
@Katherine-Osos
Copy link
Contributor Author

@rachidatecs @CocoByte All the words in that message are pretty important, but I think we only need to reduce a few characters, right? Can you try this (shortened by 3 characters)

This domain is administratively paused. It can’t be edited and won’t resolve in DNS. Contact help@get.gov for details.

@abroddrick
Copy link
Contributor

@Katherine-Osos unfortunately, shortening it by 3 characters could still result in it being cut off on some screen sizes. To avoid it ever being cut off the ideal state would be for it to be just a few words, such as "needs name servers".

@Katherine-Osos
Copy link
Contributor Author

@abroddrick The tooltip resizes appropriately on most screensizes, it's just the version I posted above that gets cut off and it's two letters. Can a dev give it a try before we make any more adjustments?

@vickyszuchin
Copy link

Moving this ticket from sprint 55 to the current sprint (sprint 56). cc: @abroddrick @lizpearl

@katypies katypies closed this as completed by moving to ✅ Done in .gov Product Board Oct 21, 2024
@vickyszuchin vickyszuchin added the carryover Carryover from a previous sprint label Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that isn't working as intended carryover Carryover from a previous sprint dev Feature: 🏢 Org Model
Projects
Status: ✅ Done
Development

No branches or pull requests

6 participants