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

CMD-111: Tomas: New Mobile Table UI is Not Backwards Compatible #319

Merged

Conversation

R-Tomas-Gonzalez
Copy link
Collaborator

@R-Tomas-Gonzalez R-Tomas-Gonzalez commented Mar 12, 2024

Overview

Remove the necessity for the .has-data-cols class. Use the new :has pseudo-selector instead.

Related

https://tacc-main.atlassian.net/browse/CMD-111

Changes

Removes the necessity for the .has-data-cols class. Uses the new :has pseudo-selector instead.

Testing

  1. Check all of the tables that have td[data-col] in the tbody.
  2. Check them again without data-col in the markup.
    Run gcp -n d564bad2712b3a96e0461ea620df5d572ddce42a. Restart demo.

UI

Screenshot 2024-03-12 at 5 18 25 PM
Desktop Mobile
Screenshot 2024-03-12 at 5 27 35 PM Screenshot 2024-03-12 at 5 27 45 PM
Desktop Mobile
Screenshot 2024-03-12 at 5 28 00 PM Screenshot 2024-03-12 at 5 27 53 PM
Desktop Mobile
Screenshot 2024-03-12 at 5 28 07 PM Screenshot 2024-03-12 at 5 28 14 PM

@github-actions github-actions bot added the feature A new feature or replacement of existing feature label Mar 12, 2024
@wesleyboar wesleyboar added the patch A backward-compatible fix label Mar 12, 2024
Copy link
Member

@wesleyboar wesleyboar left a comment

Choose a reason for hiding this comment

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

It is backwards-compatible. Thanks!

I noticed the demo shows empty space for header column on the no header table, but I'll try to resolve that separately.

@wesleyboar wesleyboar merged commit 45ee747 into feat/cmd-111-new-mobile-table-ui Mar 15, 2024
@wesleyboar wesleyboar deleted the feat/cmd-111-selector-fix branch March 15, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature or replacement of existing feature patch A backward-compatible fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants