search

cisagov / manage.get.gov

A Django-based domain name registrar used by the .gov domain to communicate with an EPP registry
https://get.gov
Other
59 stars 18 forks source link

List markup in /admin is incorrect #2232

Open rachidatecs opened 4 months ago

rachidatecs commented 4 months ago

Current Behavior

1- List markup for other websites is currently anchors with empty divs used for spacing:

Screenshot 2024-05-29 at 11 43 40 AM

2- List markup for alternative domains is currently anchors with separating commas:

Screenshot 2024-05-29 at 11 42 04 AM

Expected Behavior

1- Markup should be an unordered list with CSS on li to adjust spacing 2- Markup should be an unordered list with CSS on ul/li to display list items inline with the comma separators as pseudo class CSS content

Steps to Reproduce

  1. Inspect items above in dev tools

Environment

No response

Additional Context

No response

Issue Links

No response

PaulKuykendall commented 2 months ago

@rachidatecs - is this important? sounds like a quickfix, true?