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

card-group: See the purpose of cardGroupItemSelector static property #4144

Closed
asudoh opened this issue Oct 6, 2020 · 0 comments
Closed

card-group: See the purpose of cardGroupItemSelector static property #4144

asudoh opened this issue Oct 6, 2020 · 0 comments
Assignees
Labels
dev Needs some dev work package: web components Work necessary for the IBM.com Library web components package priority: high
Milestone

Comments

@asudoh
Copy link
Contributor

asudoh commented Oct 6, 2020

The problem

<dds-card-group> has cardGroupItemSelector static property. However, it doesn't seem to be used anywhere.

The solution

See if there is an actual use case. If so, rename it to selectorItem to align the naming convention to other components. Otherwise, remove the static property.

@asudoh asudoh added dev Needs some dev work package: web components Work necessary for the IBM.com Library web components package labels Oct 6, 2020
@RobertaJHahn RobertaJHahn added this to the Sprint 20-25 milestone Oct 14, 2020
asudoh added a commit to asudoh/ibm-dotcom-library that referenced this issue Dec 8, 2020
Change the static properties for the CSS selectors, so it aligns to the
rest of the codebase (except `stableSelector` for name-compatibility
reasons with React).

Also removes `sameHeight()` JavaScript code as it'll be replaced with

Refs carbon-design-system#4193.
Refs carbon-design-system#4144.
kodiakhq bot pushed a commit that referenced this issue Dec 9, 2020
### Related Ticket(s)

Refs #4193.
Refs #4144.

### Description

Change the static properties for the CSS selectors, so it aligns to the rest of the codebase (except `stableSelector` for name-compatibility reasons with React).

Also removes `sameHeight()` JavaScript code as it'll be replaced with #4657.

### Changelog

**Changed**

- Change the static properties for the CSS selectors, so it aligns to the rest of the codebase (except `stableSelector` for name-compatibility reasons with React).

**Removed**

- `sameHeight()` JavaScript code as it'll be replaced with #4657.
@asudoh asudoh self-assigned this Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev Needs some dev work package: web components Work necessary for the IBM.com Library web components package priority: high
Projects
None yet
Development

No branches or pull requests

2 participants