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

issue #8275, cta style/contentBlockSimple #11055

Closed
wants to merge 2 commits into from

Conversation

yafira
Copy link

@yafira yafira commented Oct 19, 2023

Related Ticket(s)

Closes #8275

Description

I looked into #8275 and it seems that the issue reporter was wrong and indeed if contentBlockSimple component is given the cta-style prop with value text, the text now shows as well as an icon. There isn't a bug and I am making a change so that the next person doesn't get confused; I created a new knob cta-text, gave the component a new prop (copy) in the story, and modified the value of an existing prop (heading) in the story.

Screenshot 2023-11-01 at 12 47 35 PM

Changelog

Changed

  • created a new knob cta-text
  • modified an existing prop heading
  • gave the component a new prop (copy) in the story

@yafira yafira requested a review from a team as a code owner October 19, 2023 21:47
@yafira yafira requested review from emyarod and annawen1 October 19, 2023 21:47
@github-actions
Copy link
Contributor

github-actions bot commented Oct 19, 2023

DCO Assistant Lite bot All Contributors have signed the CLA.

@yafira
Copy link
Author

yafira commented Oct 19, 2023

I have read the DCO document and I hereby sign the DCO.

@yafira
Copy link
Author

yafira commented Oct 19, 2023

recheck

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Oct 19, 2023

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Oct 19, 2023

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Oct 19, 2023

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Oct 19, 2023

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Oct 19, 2023

@kennylam
Copy link
Member

@yafira Unfortunately the @carbon/ibmdotcom-react package has been deprecated and no longer supported. Thanks so much for your contribution!

@kennylam kennylam closed this Mar 21, 2024
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.

[contentBlockSimple]: cta-style=text only shows icon
3 participants