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

Update: Modify Image Web Component Styles #27567

Merged
merged 4 commits into from
Apr 19, 2023
Merged

Update: Modify Image Web Component Styles #27567

merged 4 commits into from
Apr 19, 2023

Conversation

halerankin
Copy link
Contributor

Updates story with better placeholder images. Sets default border-radius and updates shape param description in the story with detail about the default border radius and available values to choose from when implementing.

Please verify that:

  • Code is up-to-date with the web-components-v3 branch
  • You've run yarn change locally

Previous Behavior

When shape: rounded is selected, no border-radius is applied.
shape param description in the story does not provide the developer with border-radius implementation details.

New Behavior

Border radius styles were added, using the 4px default value for when shape: rounded is selected.
The shape: rounded description in the story now has implementation details for the developer.

Related Issue(s)

…e that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.
…spec. Updates attribute descriptions in the story.
@fabricteam
Copy link
Collaborator

fabricteam commented Apr 13, 2023

📊 Bundle size report

🤖 This report was generated against bdb9124866e9a0c64b5a609826a9178c18e7e1ad

@size-auditor
Copy link

size-auditor bot commented Apr 13, 2023

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: bdb9124866e9a0c64b5a609826a9178c18e7e1ad (build)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 13, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit dffcee3:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@halerankin halerankin merged commit 9cd641f into microsoft:web-components-v3 Apr 19, 2023
chrisdholt pushed a commit that referenced this pull request Apr 29, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 29, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 29, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 30, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request Apr 30, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 2, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 2, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 2, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 3, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 6, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 6, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 8, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
radium-v pushed a commit that referenced this pull request May 10, 2024
* Updates story with better placeholder images. Updates styles to ensure that shadow, shape and border work correctly when fit is center. Sets default border-radius and updates storybook shape description with detail about the default border radius.

* Yarn change

* Reverts style changes. Adds only essential style adjustments to meet spec. Updates attribute descriptions in the story.

* Removes pixel value in description for design token: borderRadiusMedium.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants