react icon indicating copy to clipboard operation
react copied to clipboard

Published 20 hours ago • verified publisher icon primer

Flash has auto styling that interfere with SVG elements

Open liranelisha opened this issue 1 year ago • 11 comments

Description

I am writing to report an issue with a Flash component that I have been using on my website. The component appears to have an auto-styling feature that is interfering with the SVG elements on my web pages.

Specifically, the auto-styling feature is causing the SVG elements to be displayed incorrectly with fixed marginRight of 2 (8px). This is an issue, as the SVG elements are a part of the design and functionality of my pages.

I have tried adjusting the CSS styles for the SVG elements to no avail, as the auto-styling seems to get a priority. The only way to resolve the issue seems to be to add the !important.

I would greatly appreciate it if you could look into this issue and provide a solution as soon as possible. If necessary, I can provide you with access to my website and the code for the Flash component and the SVG elements.

Thank you for your prompt attention to this matter.

image

Steps to reproduce

create a StyledOcticon inside a Flash component

Version

35.21.0

Browser

No response

liranelisha avatar Mar 13 '23 15:03 liranelisha

Here is another reproduction: image

dejnon avatar Mar 13 '23 15:03 dejnon

Hi! This issue has been marked as stale because it has been open with no activity for 180 days. You can comment on the issue or remove the stale label to keep it open. If you do nothing, this issue will be closed in 7 days.

github-actions[bot] avatar Oct 08 '23 16:10 github-actions[bot]

this issue was seen in the devportal example: https://devportal.githubapp.com/devportal/apps/github?tab=deployments when we have the "external link" icon when the text include a link to external source

liranelisha avatar Oct 17 '23 08:10 liranelisha

Thanks for the report, this does seem to be a bug.

Internal note: I am removing the react label from this issue for now so that Primer Design can hopefully prioritize this. @tallys let us know if that's not doable.

lesliecdubs avatar Oct 23 '23 21:10 lesliecdubs

@lukasoppermann let's take a look at this as part of our work on flash/banner

tallys avatar Oct 24 '23 14:10 tallys

Yes. 👍

lukasoppermann avatar Oct 24 '23 15:10 lukasoppermann

Hi! This issue has been marked as stale because it has been open with no activity for 180 days. You can comment on the issue or remove the stale label to keep it open. If you do nothing, this issue will be closed in 7 days.

github-actions[bot] avatar Apr 21 '24 16:04 github-actions[bot]

sill happening... any updates @lukasoppermann ?

liranelisha avatar Apr 24 '24 11:04 liranelisha

@liranelisha we will keep this in mind for the new component.

CC: @JoseInTheArena should we add the react to make sure it is address in #4335 ?

lukasoppermann avatar Apr 24 '24 11:04 lukasoppermann