Skip to content

Fix hex conversion with alpha#511

Merged
GuillaumeGomez merged 2 commits into
masterfrom
hex-conversion
May 29, 2023
Merged

Fix hex conversion with alpha#511
GuillaumeGomez merged 2 commits into
masterfrom
hex-conversion

Conversation

@GuillaumeGomez

Copy link
Copy Markdown
Owner

Funny case where rgba(255, 236, 164, 0.06) was converted into #ffeca4f.4ccccccccccc8. A rounding was missing.

@GuillaumeGomez GuillaumeGomez merged commit ba049dd into master May 29, 2023
@GuillaumeGomez GuillaumeGomez deleted the hex-conversion branch May 29, 2023 13:55
Noratrieb added a commit to Noratrieb/rust that referenced this pull request May 30, 2023
…lor-9, r=notriddle

Migrate GUI colors test to original CSS color format

Follow-up of rust-lang#111459.

The `browser-ui-test` update is a fix when converting the alpha value to hex format. More information [here](GuillaumeGomez/browser-UI-test#511).

r? `@notriddle`
Noratrieb added a commit to Noratrieb/rust that referenced this pull request May 30, 2023
…lor-9, r=notriddle

Migrate GUI colors test to original CSS color format

Follow-up of rust-lang#111459.

The `browser-ui-test` update is a fix when converting the alpha value to hex format. More information [here](GuillaumeGomez/browser-UI-test#511).

r? ``@notriddle``
Noratrieb added a commit to Noratrieb/rust that referenced this pull request May 30, 2023
…lor-9, r=notriddle

Migrate GUI colors test to original CSS color format

Follow-up of rust-lang#111459.

The `browser-ui-test` update is a fix when converting the alpha value to hex format. More information [here](GuillaumeGomez/browser-UI-test#511).

r? ```@notriddle```
Noratrieb added a commit to Noratrieb/rust that referenced this pull request May 30, 2023
…lor-9, r=notriddle

Migrate GUI colors test to original CSS color format

Follow-up of rust-lang#111459.

The `browser-ui-test` update is a fix when converting the alpha value to hex format. More information [here](GuillaumeGomez/browser-UI-test#511).

r? ````@notriddle````
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.

1 participant