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

An element with display: inline-block will cause a containing span's border to be cut off. #13142

Closed
Permutatrix opened this issue Aug 31, 2016 · 4 comments
Closed

An element with display: inline-block will cause a containing span's border to be cut off. #13142

Permutatrix opened this issue Aug 31, 2016 · 4 comments
Labels
A-layout/inline I-wrong

Comments

@Permutatrix
Copy link
Contributor

@Permutatrix Permutatrix commented Aug 31, 2016 •
edited

While trying to decipher the CSS specification with regard to #12939, I created a document that looked something like this to see how Firefox and Chromium would interpret it:

<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8" />
  </head>
  <body>
    <span id="outer" style="border: 1px solid #000">
      Test 3
      <span id="inner" style="display: inline-block; overflow: hidden; width: 100px; height: 100px">XYZ</span>
    </span>
  </body>
</html>

...which is rendered by Firefox like this:
Firefox's correct rendering

...by Chromium like this:
Chromium's correct rendering

...and by Servo like this:
Servo's incorrect rendering
@Permutatrix Permutatrix mentioned this issue Aug 31, 2016
Open
@notriddle
Copy link
Contributor

@notriddle notriddle commented Sep 4, 2016

Duplicate of #7681?
@Permutatrix
Copy link
Contributor Author

@Permutatrix Permutatrix commented Sep 4, 2016

@notriddle That seems likely. I didn't check for duplicates because I wasn't sure how and figured someone else would catch it, like you just did. I hope that was proper etiquette.
@Permutatrix
Copy link
Contributor Author

@Permutatrix Permutatrix commented Sep 4, 2016

@notriddle I removed the is_primary_fragment condition you mentioned in this comment on that issue. It now renders the right half of the border, but it's much too large and disconnected from the left half:
sr

This seems to match the results you described.
@nox
Copy link
Member

@nox nox commented Oct 4, 2017

Closing as duplicate of #7681.
@nox nox closed this Oct 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-layout/inline I-wrong
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@nox @emilio @notriddle @Permutatrix
You can’t perform that action at this time.