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

feat: des-2697 add c-app-card, improve s-app-card #318

Merged
merged 9 commits into from
Mar 15, 2024

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Mar 12, 2024

Overview / Changes

Add c-app-card (based off of s-cap-card). Slight improvements to s-app-card.

Status

  • create c-app-card
  • test c-app-card
  • get user to review markup
    if review suggests changes, those can be made
  • screenshot UI
  • delete s-app-card
    I do this in a separate PR. I want to commit the improvements here, before deleting the pattern. If it needs to be restored, restoring will happen with all improvements.

Related

Testing

  1. Open http://localhost:3000/components/preview/s-app-card.
  2. See how un-skinned it looks.
  3. But notice it has a structure / layout.
  4. Compare to skinned version.

UI

c-app-card s-app-card
c-app-card s-app-card

@github-actions github-actions bot added the feature A new feature or replacement of existing feature label Mar 12, 2024
@wesleyboar wesleyboar added the minor A feature in backward-compatible manner label Mar 12, 2024
@wesleyboar wesleyboar marked this pull request as ready for review March 15, 2024 20:04
@wesleyboar wesleyboar merged commit 1b1e800 into main Mar 15, 2024
@wesleyboar wesleyboar deleted the feat/DES-2697-c-app-card-from-s-cap-card branch March 15, 2024 20:04
@wesleyboar wesleyboar changed the title feat: des-2697 add c-app-card, delete s-cap-card feat: des-2697 add c-app-card Mar 15, 2024
@wesleyboar wesleyboar changed the title feat: des-2697 add c-app-card feat: des-2697 add c-app-card, improve s-app-card Mar 15, 2024
wesleyboar added a commit that referenced this pull request Mar 15, 2024
* refactor: DES-2697 s-app-card selector rename

* refactor: DES-2697 c-app-card (clones s-app-card)

* refactor: DES-2697 c-app-card uses class names

* fix: DES-2697 support a.c-app-card

* style: DES-2697 remove whitespace in cond. class

* style: DES-2697 remove extra spaces

* docs: DES-2697 update comments, new element names

* docs: DES-2697 only use trick link css when link exists

* fix: DES-2697 do not let client overwrite transparent trick link
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 minor A feature in backward-compatible manner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant