servo icon indicating copy to clipboard operation
servo copied to clipboard

Update the rendering: consolidate reflows

Open gterzian opened this issue 2 years ago • 2 comments

Follow-up to https://github.com/servo/servo/issues/31242

Step 20 of update-the-rendering says to "update the rendering or user interface of doc and its node navigable to reflect the current state".

I presume that, in Servo, this means issuing a reflow, which is done at https://github.com/servo/servo/pull/31505/files#diff-2c5948584719622eef023c56cdf90e8278971f2f2eda26e7fd0485a24c32be3cR1794

However, there are many calls to reflow made across script. I think it needs a separate PR to track them all down and decide which ones are necessary(but probably not spec conformant), and which ones can be removed(or rather consolidated into that one step 20).

gterzian avatar Mar 26 '24 02:03 gterzian

Would like to work on this. Can you please assign this to me @gterzian

nishchay-veer avatar Apr 02 '24 14:04 nishchay-veer

Thank you for your interest! This is depends on an open PR, and it is also very complicated. If this is your first contribution I recommend looking for issues with tags B-newcomer(of which there might not be many now, sorry for that).

gterzian avatar Apr 03 '24 05:04 gterzian