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: add styles for components #24

Merged
merged 4 commits into from
Nov 29, 2023
Merged

Conversation

ninamarina
Copy link
Contributor

@ninamarina ninamarina commented Nov 27, 2023

Part of the effort to remove all utility classes from Dialtone Vue.

Jira: https://dialpad.atlassian.net/browse/DLT-1125
Subtask: https://dialpad.atlassian.net/browse/DLT-1387
Dialtone vue: dialpad/dialtone-vue#1314

Description

Adds styles for the following components in dialtone library:

Pagination
Popover
Radio Checkbox
Skeleton
Validation Message

This would complete all components for the task https://dialpad.atlassian.net/browse/DLT-1125

Pull Request Checklist

  • Ask the contributors if a similar effort is already in process or has been solved.
  • Review the contribution guidelines.
  • Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
  • Ensure all gulp scripts successfully compile.
  • Update, remove, or extend all affected documentation.
  • Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

gosling-barbie

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-24
🔨 If you experience an SSL issue then wait 2 minutes and try again.

1 similar comment
Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-24
🔨 If you experience an SSL issue then wait 2 minutes and try again.

display: flex;
flex-direction: row;
grid-gap: var(--dt-space-300);
align-items: center;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-d-flex d-fd-row d-gg4 d-ai-center

.d-pagination__button {
padding-right: var(--dt-space-400);
padding-left: var(--dt-space-400);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-px8

// ----------------------------------------------------------------------------
.d-pagination__separator {
align-self: flex-end;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-as-flex-end

width: var(--dt-size-550);
color: var(--dt-color-foreground-tertiary);
text-align: center;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-fc-tertiary d-w24 d-ta-center

&__content {
width: 100%;
text-overflow: ellipsis;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-to-ellipsis, d-w100p

margin-right: var(--dt-space-350);
padding: var(--dt-space-350);
border-color: transparent;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-p6 d-mr6 d-bc-transparent

font-family: inherit;
line-height: var(--dt-font-line-height-400);
fill: currentColor;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-description

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, there is no changes on radio-checkbox.less

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, since d-description is not a utility I removed the change.

// @ SKELETON LIST ITEM
// ----------------------------------------------------------------------------
.d-skeleton-list-item {
display: flex;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-d-flex


&--single {
align-items: center;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-ai-center


&__shape {
margin-right: var(--dt-space-400);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-mr8

&__paragraph-container {
display: flex;
flex-direction: column;
width: 100%;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-d-flex d-fd-column d-w100p

// @ SKELETON PARAGRAPH
// ----------------------------------------------------------------------------
.d-skeleton-paragraph {
width: 100%;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-w100p


&:last-child {
margin-bottom: var(--dt-space-0);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

'd-mb12': row !== integerRows

// ----------------------------------------------------------------------------
.d-skeleton-text {
height: 80%;
border-radius: var(--dt-size-radius-200);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-h8, d-bar2

Copy link
Contributor

Choose a reason for hiding this comment

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

hmmm
height: 80%; should be height: var(--dt-size-400), what Im missing?
border-radius its ok

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are right! Bad copy pasting 🤪


.d-skeleton-text--heading {
border-radius: var(--dt-size-radius-200);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-bar2

// $ BASE STYLE
// ----------------------------------------------------------------------------
.d-tab-panel--hidden {
display: none;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-d-none

// ----------------------------------------------------------------------------
.d-toggle-wrapper {
display: flex;
align-items: center;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-d-flex d-ai-center


&__container {
display: flex;
flex-direction: column;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

d-d-flex d-fd-column

Copy link

✔️ Dialtone Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/deploy-previews/pr-24

Copy link

✔️ Dialtone-vue 3 Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-24

Copy link

✔️ Dialtone Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/deploy-previews/pr-24

@ninamarina ninamarina force-pushed the add/componets-styles branch 3 times, most recently from 59246a5 to ef27ab9 Compare November 28, 2023 16:35
Copy link

✔️ Dialtone-vue 3 Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-24

@dialpad dialpad deleted a comment from github-actions bot Nov 28, 2023
@dialpad dialpad deleted a comment from github-actions bot Nov 28, 2023
@dialpad dialpad deleted a comment from github-actions bot Nov 28, 2023
@dialpad dialpad deleted a comment from github-actions bot Nov 28, 2023
@dialpad dialpad deleted a comment from github-actions bot Nov 28, 2023
@dialpad dialpad deleted a comment from github-actions bot Nov 28, 2023
@ninamarina ninamarina marked this pull request as ready for review November 28, 2023 18:44
Copy link

✔️ Dialtone Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/deploy-previews/pr-24

Copy link

✔️ Dialtone-vue 3 Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-24

Copy link
Contributor

@iropolo iropolo left a comment

Choose a reason for hiding this comment

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

Looks good, just one coment about d-h8 on skeleton.less.

Thanks for doing the comments, thats helps a lot the review process

Copy link

✔️ Dialtone Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/deploy-previews/pr-24

Copy link

✔️ Dialtone-vue 3 Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-24

@ninamarina ninamarina merged commit 66ba329 into staging Nov 29, 2023
5 checks passed
@ninamarina ninamarina deleted the add/componets-styles branch November 29, 2023 13:09
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