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

problem displaying the ripple on the outlined button #5620

Closed
Benoit-ROBIN opened this issue May 16, 2024 · 2 comments · Fixed by #5623
Closed

problem displaying the ripple on the outlined button #5620

Benoit-ROBIN opened this issue May 16, 2024 · 2 comments · Fixed by #5623

Comments

@Benoit-ROBIN
Copy link
Contributor

Benoit-ROBIN commented May 16, 2024

What is affected?

Component, Theming

Description

When I try to change the style of md-outlined-button I have displaying issues.
When I change this value :

.button-outlined {
  --md-ripple-pressed-opacity: 1;
}

there is a white border inside the button
Capture d’écran 2024-05-14 à 17 09 00

The problem is because of mismatching border lines, which causes the ripple's border to not perfectly line up with a border on its parent.

Workaround

A workaround was found by overriding the ripple border-radius value:

.button-outlined::part(ripple) {
  border-radius: calc(var(--eds-button-outlined-border-radius) - 1px);
}

to do this I need to add the attribute part to md-ripple in the button component.
source: https://discord.com/channels/259087343246508035/1239956071762624563

Is this a regression?

No or unsure. This never worked, or I haven't tried before.

Affected versions

1.4.1

Browser/OS/Node environment

node: 18.12.1
npm: 8.19.2
chrome: 124.0.6367.208
macos: 14.2.1

@Benoit-ROBIN
Copy link
Contributor Author

I have a local branch ready but I can't push it to create a pull request.

ERROR: Permission to material-components/material-web.git denied to Benoit-ROBIN.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

@asyncLiz
Copy link
Collaborator

You won't be able to push your branch directly to this repository. Instead, create a pull request from a cloned copy of this repository.

GitHub has some guides on creating PRs from forks.

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 a pull request may close this issue.

2 participants