Skip to content

/ servo

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

Jump to bottom

CSS properties are being applied twice for text node inside Flexbox #19793

Open
ocerman opened this issue Jan 17, 2018 · 2 comments
Open

CSS properties are being applied twice for text node inside Flexbox #19793

ocerman opened this issue Jan 17, 2018 · 2 comments
Labels
C-has-manual-testcase

Comments

@ocerman
Copy link

@ocerman ocerman commented Jan 17, 2018

servo

<!DOCTYPE html>
<html>
	<head>
		<style>
			html{
				background: #BADA55;
			}
			.item{
				margin: 10px;
				height: 40px;
				color: white;
				display: inline-flex;
				border: 2px solid cyan;
				
				background: rgba(0, 0, 0, 0.7);
				transform: rotateZ(-40deg) translate(0px, 80px) scale(1.5);
				filter: invert(100%);
				font-size: 2em;
			}
		</style>
	</head>
	<body>
		<div class="item">
			Servo
		</div>
	</body>
</html>
@ocerman ocerman changed the title CSS properties are being applied two times for text node inside Flexbox CSS properties are being applied twice for text node inside Flexbox Jan 17, 2018
@jdm
Copy link
Member

@jdm jdm commented Jan 17, 2018

This sounds like #11300, which was supposed to be fixed by #14078 but which was never merged.
@ocerman
Copy link
Author

@ocerman ocerman commented Jan 18, 2018

@jdm yes, it sounds similiar, however, I tried all test cases in #11300 and they were rendered correctly. This issue seems to be connected to flexbox, because it is happening when using display: flex or display:inline-flex.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-has-manual-testcase
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@jdm @ocerman
You can’t perform that action at this time.