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

(chore) Update Carbon #17

Closed
wants to merge 3 commits into from
Closed

Conversation

Vijaykv5
Copy link
Contributor

Requirements

  • This PR has a title that briefly describes the work done including a conventional commit type prefix and a Jira ticket number if applicable. See existing PR titles for inspiration.

For changes to apps

If applicable

  • My work includes tests or is validated by existing tests.

Summary

This PR updates the carbon version to the latest, updates carbon styling imports and removes carbon from declarations.

Screenshots

Related Issue

Other

@Vijaykv5 Vijaykv5 changed the title (chore) : Update Carbon (chore) Update Carbon Aug 15, 2024
@Vijaykv5 Vijaykv5 marked this pull request as draft August 15, 2024 14:42
@Vijaykv5
Copy link
Contributor Author

@NethmiRodrigo May you can check this out?

@@ -1,21 +1,21 @@
@use '@carbon/styles/scss/spacing';
@use '@carbon/layout';
@use '@carbon/styles/scss/type';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
@use '@carbon/styles/scss/type';
@use '@carbon/type';

@@ -1,23 +1,23 @@
@use '@carbon/styles/scss/spacing';
@use '@carbon/layout';
@use '@carbon/styles/scss/type';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
@use '@carbon/styles/scss/type';
@use '@carbon/type';

@NethmiRodrigo
Copy link

You're right with the carbon upgrade not fixing the close button rendering properly on the modal, I'm not quite sure yet why... Please give me a bit more time to check

@Vijaykv5
Copy link
Contributor Author

You're right with the carbon upgrade not fixing the close button rendering properly on the modal, I'm not quite sure yet why... Please give me a bit more time to check

Sure...Thanks @NethmiRodrigo

@Vijaykv5
Copy link
Contributor Author

@NethmiRodrigo Have you checked it out?

@Vijaykv5
Copy link
Contributor Author

Vijaykv5 commented Oct 5, 2024

This has been fixed over #20

@Vijaykv5 Vijaykv5 closed this Oct 5, 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.

2 participants