Skip to content

Conversation

Harshit11Narula
Copy link

No description provided.

Copy link
Collaborator

@t007rushi t007rushi left a comment

Choose a reason for hiding this comment

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

There is no need of self explanotory comments we can remove to reduce lib size

},
methods: {
getLabelStyle(crumb) {
// Merge the default labelStyle with crumb-specific styles, if any
Copy link
Collaborator

Choose a reason for hiding this comment

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

if comments use is complete can you remove plz

}

.image-container.rounded-corner img {
border-radius: 8px; /* Set your desired border radius */
Copy link
Collaborator

Choose a reason for hiding this comment

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

try to use consistent units as above used 0.5 rem and here 8px
as both same on the calculation
will suggest to use relative units

Comment on lines +26 to +38
.jds-link:hover {
color: red; /* Change link color on hover */
text-decoration: none; /* Remove underline on hover */
}

/* Active styles for the link (when clicked) */
.jds-link:active {
color: green; /* Change link color when clicked */
}

/* Focus styles for the link (when tabbed to) */
.jds-link:focus {
outline: 2px solid purple; /* Add an outline when the link is focused */
Copy link
Collaborator

Choose a reason for hiding this comment

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

no need for comments I think as the selectors are self explanatory

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