Skip to content

4238 lang attribute input label#4261

Draft
jd3267 wants to merge 6 commits intodevelopfrom
4238-lang-attribute-input-label
Draft

4238 lang attribute input label#4261
jd3267 wants to merge 6 commits intodevelopfrom
4238-lang-attribute-input-label

Conversation

@jd3267
Copy link
Contributor

@jd3267 jd3267 commented Feb 11, 2026

Summary of the changes

Added label slot to ic-input-label to allow for addition of lang attributes to text

Related issue

#4238

Checklist

General

  • Changes to docs package checked and committed.
  • All acceptance criteria reviewed and met.

@github-actions
Copy link
Contributor

@jd3267 jd3267 force-pushed the 4238-lang-attribute-input-label branch from a501385 to 3518e86 Compare February 11, 2026 14:08
@jd3267 jd3267 marked this pull request as ready for review February 11, 2026 14:09
@lz405
Copy link
Contributor

lz405 commented Feb 11, 2026

Could use a storybook example of using the new slot.

@GCHQ-Developer-530
Copy link
Contributor

Could use a storybook example of using the new slot.

And a Cypress visual regression test too please

@jd3267 jd3267 marked this pull request as draft February 12, 2026 09:53
@jd3267 jd3267 force-pushed the 4238-lang-attribute-input-label branch 2 times, most recently from 59210cd to 5239443 Compare February 16, 2026 14:29
update to docs for adding of label slot to ic-input-label
…t-label

update to docs for adding of label slot to ic-input-label
addition of label slot to allow adding lang attributes to text
@jd3267 jd3267 force-pushed the 4238-lang-attribute-input-label branch from 5239443 to 14332d8 Compare February 16, 2026 14:30
adds story example to checkbox-group to test slotted label with changes to input-label
add french words to cspell for language test
@jd3267 jd3267 force-pushed the 4238-lang-attribute-input-label branch from 14332d8 to 67c5e9f Compare February 16, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants