1896210 - Revise layout overview doc

Status: RESOLVED FIXED

(Core :: Layout, task)

Description

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

See the upcoming patches.

Comment 1

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1896210 Part 1 - Minor format tweak for LayoutOverview.md. r?dholbert,#layout

Drop backslashes because escaping quote, double-quote, etc. is not needed in
markdown. Indent fix for lists.

Comment 2

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1896210 Part 2 - Revise LayoutOverview.md. r?dholbert,#layout

• Delete paragraph starting with "The frame tree should not store any important
  data .." because I don't understand what specifically it wants to convey.
• Delete XUL related content.
• Add a section "Physical Coordinates vs Logical Coordinates"

Note: ./mach doc layout/docs generates the doc preview locally.

Comment 3

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1896210 Part 3 - Update comments pointing to new layout overview doc. r?dholbert,#layout

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/03f6faac8436
Part 1 - Minor format tweak for LayoutOverview.md. r=layout-reviewers,emilio
https://hg.mozilla.org/integration/autoland/rev/532d6a9a5af0
Part 2 - Revise LayoutOverview.md. r=dshin
https://hg.mozilla.org/integration/autoland/rev/160a8086a2f2
Part 3 - Update comments pointing to new layout overview doc. r=dholbert

Comment 5

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1896210 Part 4 - Address late review comments for Part 2. r?dholbert

Comment 6

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/03f6faac8436
https://hg.mozilla.org/mozilla-central/rev/532d6a9a5af0
https://hg.mozilla.org/mozilla-central/rev/160a8086a2f2

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/58956567e704
Part 4 - Address late review comments for Part 2. r=dholbert DONTBUILD

Comment 8

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/58956567e704