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

refactor ProgressRadial to use pseudo-elements #1356

Merged
merged 31 commits into from
Jun 16, 2023
Merged
Show file tree
Hide file tree
Changes from 24 commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
c831537
wip
mayank99 Jun 6, 2023
50a4f22
Merge branch 'dev' into mayank/spinner
mayank99 Jun 7, 2023
67646ae
Merge branch 'dev' into mayank/spinner
mayank99 Jun 14, 2023
7181888
fix everything i think
mayank99 Jun 14, 2023
162257d
typo
mayank99 Jun 14, 2023
4d9d325
remove garbage
mayank99 Jun 14, 2023
47e8652
font size fix
mayank99 Jun 14, 2023
5f42edc
jsdoc
mayank99 Jun 14, 2023
99fbd01
font fix again
mayank99 Jun 14, 2023
1c60e8c
html cleanup
mayank99 Jun 14, 2023
90ffe23
border fix
mayank99 Jun 14, 2023
3625662
border fix again
mayank99 Jun 14, 2023
bfa7934
remove iui-inner-content
mayank99 Jun 15, 2023
57681e0
handle min/max percentages
mayank99 Jun 15, 2023
cd911ce
update some tests
mayank99 Jun 15, 2023
daa553b
typo
mayank99 Jun 15, 2023
0391b75
fix checkbox test
mayank99 Jun 15, 2023
78f95c5
update other components
mayank99 Jun 15, 2023
8e5ace9
update story
mayank99 Jun 15, 2023
f1b4733
update remaining images
mayank99 Jun 15, 2023
b797203
changesets
mayank99 Jun 15, 2023
8ef55b6
fix scroll snapping
mayank99 Jun 15, 2023
5f234f9
update image again
mayank99 Jun 15, 2023
ddfbcab
Merge branch 'dev' into mayank/spinner
mayank99 Jun 16, 2023
d92c7cd
fix changeset
mayank99 Jun 16, 2023
55e448e
use border-radius variable
mayank99 Jun 16, 2023
529e993
change easing
mayank99 Jun 16, 2023
d01803f
fix forced-colors mode
mayank99 Jun 16, 2023
404e52a
revert direction of gradient
mayank99 Jun 16, 2023
d6f5ac1
Merge branch 'dev' into mayank/spinner
mayank99 Jun 16, 2023
e3ef3ad
update wrong comments
mayank99 Jun 16, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 6 additions & 0 deletions .changeset/quick-balloons-drum.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
'@itwin/itwinui-react': major
'@itwin/itwinui-css': major
gretanausedaite marked this conversation as resolved.
Show resolved Hide resolved
---

ProgressRadial has been refactored to be a single `<div>` instead of using a nested svg. Also it is recommended to explicitly set `size` when using in other components.
5 changes: 5 additions & 0 deletions .changeset/sharp-oranges-kneel.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@itwin/itwinui-css': major
---

`iui-progress-radial` has been refactored to be a single `<div>` instead of using a nested svg. Size and status have been moved to data attributes. Also it is recommended to explicitly set `size` when using in other components.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 0 additions & 2 deletions apps/storybook/src/ProgressRadial.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ export default {
argTypes: {
className: { control: { disable: true } },
style: { control: { disable: true } },
title: { control: { disable: true } },
id: { control: { disable: true } },
},
args: {
value: 50,
Expand Down
2 changes: 1 addition & 1 deletion apps/storybook/src/Table.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4041,7 +4041,7 @@ export const StatusAndCellIcons: Story<Partial<TableProps>> = (args) => {
startIcon={props.cellProps.row.original.startIcon}
endIcon={
props.cellProps.row.original.isLoading ? (
<ProgressRadial value={40} />
<ProgressRadial value={40} size='small' />
) : (
props.cellProps.row.original.endIcon
)
Expand Down
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
50 changes: 8 additions & 42 deletions packages/itwinui-css/backstop/tests/checkbox.html
Original file line number Diff line number Diff line change
Expand Up @@ -570,27 +570,10 @@ <h3>Checkbox on left</h3>
disabled
tabindex="-1"
/>
<div class="iui-progress-indicator-radial iui-indeterminate iui-x-small">
<svg
class="iui-radial"
viewBox="0 0 32 32"
aria-hidden="true"
>
<circle
class="iui-track"
cx="16"
cy="16"
r="14"
>
</circle>
<circle
class="iui-fill"
cx="16"
cy="16"
r="14"
>
</circle>
</svg>
<div
class="iui-progress-indicator-radial iui-indeterminate"
data-iui-size="x-small"
>
</div>
<span class="iui-checkbox-label">Loading checkbox</span>
</label>
Expand Down Expand Up @@ -684,27 +667,10 @@ <h3>Checkbox on right</h3>
disabled
tabindex="-1"
/>
<div class="iui-progress-indicator-radial iui-indeterminate iui-x-small">
<svg
class="iui-radial"
viewBox="0 0 32 32"
aria-hidden="true"
>
<circle
class="iui-track"
cx="16"
cy="16"
r="14"
>
</circle>
<circle
class="iui-fill"
cx="16"
cy="16"
r="14"
>
</circle>
</svg>
<div
class="iui-progress-indicator-radial iui-indeterminate"
data-iui-size="x-small"
>
</div>
</label>

Expand Down
20 changes: 0 additions & 20 deletions packages/itwinui-css/backstop/tests/non-ideal-state.html
Original file line number Diff line number Diff line change
Expand Up @@ -2647,26 +2647,6 @@
<div class="iui-non-ideal-state">
<div class="iui-non-ideal-state-illustration">
<div class="iui-progress-indicator-radial iui-indeterminate">
<svg
class="iui-radial"
viewBox="0 0 32 32"
aria-hidden="true"
>
<circle
class="iui-track"
cx="16"
cy="16"
r="14"
>
</circle>
<circle
class="iui-fill"
cx="16"
cy="16"
r="14"
>
</circle>
</svg>
</div>
</div>
<div class="iui-non-ideal-state-title">Session timed out</div>
Expand Down