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

Components: Introduce a component for heading/copy, etc. #4326

Closed
asudoh opened this issue Oct 28, 2020 · 3 comments
Closed

Components: Introduce a component for heading/copy, etc. #4326

asudoh opened this issue Oct 28, 2020 · 3 comments
Assignees
Labels
dev Needs some dev work package: web components Work necessary for the IBM.com Library web components package Sprint Must Have
Milestone

Comments

@asudoh
Copy link
Contributor

asudoh commented Oct 28, 2020

The problem

Most of the components with heading contents in Web Components codebase has a corresponding component for heading contents, for the styling purpose. OTOH <dds-link-list> doesn't.

Also, <dds-image-with-caption> supports heading attribute but it doesn't have a corresponding <slot>.

The solution

Consider introducing a similar one for <dds-link-list>. Also scan the codebase to see if we need any more components for heading purpose, for other areas.

Additional information

Similar thing is seen with <dds-card>, etc.

@asudoh asudoh added dev Needs some dev work package: web components Work necessary for the IBM.com Library web components package labels Oct 28, 2020
@asudoh asudoh assigned kennylam, jeffchew, ghost and jacobottesen-dgc and unassigned kennylam, jeffchew, ghost and jacobottesen-dgc Oct 28, 2020
@asudoh asudoh changed the title Link list: Introduce a component for heading Link list, etc.: Introduce a component for heading Oct 28, 2020
@RobertaJHahn RobertaJHahn added this to the Sprint 21-01 milestone Oct 28, 2020
@asudoh asudoh changed the title Link list, etc.: Introduce a component for heading Components: Introduce a component for heading/copy, etc. Dec 20, 2020
asudoh added a commit to asudoh/ibm-dotcom-library that referenced this issue Dec 29, 2020
Introduces `<dds-card-eyebrow>` and `<dds-card-heading>` to make a
choice of the approach of `<div>`/`<span>` with specifying `slot`
attribute vs. dedicated components to the latter, to align to other
components. The dedicated components approach allows more control over
styling.

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

Refs #4326.

### Description

Introduces `<dds-card-eyebrow>` and `<dds-card-heading>` to make a choice of the approach of `<div>`/`<span>` with specifying `slot` attribute vs. dedicated components to the latter, to align to other components. The dedicated components approach allows more control over styling.

### Changelog

**New**

- `<dds-card-eyebrow>` and `<dds-card-heading>` as a components dedicated to card eyebrow/heading, respectively.
asudoh added a commit to asudoh/ibm-dotcom-library that referenced this issue Dec 30, 2020
Introduces `<dds-link-list-heading>` to make a choice of the approach
of `<div>`/`<span>` with specifying `slot` attribute vs. dedicated
components to the latter, to align to other components.

The dedicated components approach allows more control over styling.

Refs carbon-design-system#4326.
kodiakhq bot pushed a commit that referenced this issue Jan 4, 2021
### Related Ticket(s)

Refs #4326.

### Description

Introduces `<dds-link-list-heading>` to make a choice of the approach of `<div>`/`<span>` with specifying `slot` attribute vs. dedicated components to the latter, to align to other components.

The dedicated components approach allows more control over styling.

### Changelog

**New**

- `<dds-link-list-heading>`, the heading for link list.
asudoh added a commit to asudoh/ibm-dotcom-library that referenced this issue Jan 7, 2021
Introduces the following components:

* `<dds-callout-data-heading>`: The heading (data) content.
* `<dds-callout-data-copy>`: The copy content.
* `<dds-callout-data-source>`: The source content.

Refs carbon-design-system#4326.
kodiakhq bot pushed a commit that referenced this issue Jan 8, 2021
### Related Ticket(s)

Refs #4326.

### Description

Introduces the following components:

* `<dds-callout-data-heading>`: The heading (data) content.
* `<dds-callout-data-copy>`: The copy content.
* `<dds-callout-data-source>`: The source content.

### Changelog

**New**

- `<dds-callout-data-heading>`: The heading (data) content for `<dds-callout-data>`.
- `<dds-callout-data-copy>`: The copy content for `<dds-callout-data>`.
- `<dds-callout-data-source>`: The source content for `<dds-callout-data>`.
@RobertaJHahn
Copy link

@asudoh Can this issue be closed? I looks like all of the linked PRs are merged.

@asudoh
Copy link
Contributor Author

asudoh commented Jan 11, 2021

@RobertaJHahn Thank you for checking - Would like to keep this open if possible. Further code inspection is needed to ensure we have finished everything.

asudoh added a commit to asudoh/ibm-dotcom-library that referenced this issue Jan 13, 2021
Removes the `<div>` in `<dds-link-list>` surrounding
`<slot name="heading">`, and makes change to `<dds-link-list-heading>`
to make it work without such surrounding `<div>`.

This change ensures that `<dds-link-list>` aligns the `<slot>`
structure to other components like `<dds-card>`.

Refs carbon-design-system#4326.
kodiakhq bot pushed a commit that referenced this issue Jan 13, 2021
### Related Ticket(s)

Refs #4326.

### Description

Removes the `<div>` in `<dds-link-list>` surrounding `<slot name="heading">`, and makes change to `<dds-link-list-heading>` to make it work without such surrounding `<div>`.

This change ensures that `<dds-link-list>` aligns the `<slot>` structure to other components like `<dds-card>`.

### Changelog

**Changed**

- `<dds-link-list-heading>` to make it work without surrounding `<div>` described below.

**Removed**

- `<div>` in `<dds-link-list>` surrounding `<slot name="heading">`
@asudoh
Copy link
Contributor Author

asudoh commented Jan 14, 2021

@RobertaJHahn This work is complete now. Thanks again for checking!

IgnacioBecerra pushed a commit to IgnacioBecerra/ibm-dotcom-library that referenced this issue Feb 22, 2021
…ign-system#4887)

### Related Ticket(s)

Refs carbon-design-system#4326.

### Description

Removes the `<div>` in `<dds-link-list>` surrounding `<slot name="heading">`, and makes change to `<dds-link-list-heading>` to make it work without such surrounding `<div>`.

This change ensures that `<dds-link-list>` aligns the `<slot>` structure to other components like `<dds-card>`.

### Changelog

**Changed**

- `<dds-link-list-heading>` to make it work without surrounding `<div>` described below.

**Removed**

- `<div>` in `<dds-link-list>` surrounding `<slot name="heading">`
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 Sprint Must Have
Projects
None yet
Development

No branches or pull requests

5 participants