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

Implement `position: relative` #792

Closed
pcwalton opened this issue Aug 26, 2013 · 6 comments
Closed

Implement `position: relative` #792

pcwalton opened this issue Aug 26, 2013 · 6 comments

Comments

@pcwalton
Copy link
Contributor

@pcwalton pcwalton commented Aug 26, 2013

This is needed for Acid2 lines 10 and 11.

You should be able to implement this as a normal layout of the element followed by a translation of the generated DisplayList items.

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Dec 10, 2013

Needed for #1366.

@metajack
Copy link
Contributor

@metajack metajack commented Jan 16, 2014

mentor: @larsbergstrom

@JoonWonLee @pradeep90 @ksh8281 are working on this

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Jan 17, 2014

#1440 has landed, and should provide some code that will compute the top, left, right, and bottom properties. You should feel free to change the code as you need to make it fit this work!

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Jan 22, 2014

@JoonWonLee @pradeep90 @ksh8281 what is the status of this work? Does this issue contain sufficient information for you to make progress?

@ksh8281
Copy link
Contributor

@ksh8281 ksh8281 commented Jan 23, 2014

@larsbergstrom we are currenly working on inline elememts.
we will try position: relative soon. if we have any issue, I should pinging to lars! :)

bors-servo pushed a commit that referenced this issue Feb 5, 2014
implement position:relative

this pr implement #792
it gives same result with firefox in "src/test/ref/position_relative_a.html"
r? @larsbergstrom
@0b10011
Copy link
Contributor

@0b10011 0b10011 commented Feb 17, 2014

Shouldn't this be closed since #1613 has been merged?

@metajack metajack closed this Feb 22, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.