TPE-Layout/2018-01-09: Difference between revisions
Jump to navigation
Jump to search
Astleychen (talk | contribs) (→Weekly Report: 2018W02) |
(→Wei-Cheng Pan: ...) |
||
(9 intermediate revisions by 7 users not shown) | |||
Line 1: | Line 1: | ||
==Weekly Announcement== | ==Weekly Announcement== | ||
* CultureAMP survey will close on Friday, January 12. | |||
==Weekly Sharing== | ==Weekly Sharing== | ||
Line 5: | Line 6: | ||
==Weekly Status== | ==Weekly Status== | ||
===Astley Chen=== | ===Astley Chen=== | ||
* {{bug|1357966}} NEW:: Don't report resource timing values for image, -moz-image-rect, font, and cursor from cross-origin stylesheets | |||
** Bug 1357966 - sec-high and need work | |||
===Boris Chiou=== | ===Boris Chiou=== | ||
* [https://bugs.chromium.org/p/chromium/issues/detail?id=496518 Blink issue 496518] | |||
** Checked the current implementation of motion path on Blink | |||
* [https://drafts.fxtf.org/motion-1/ Motion Path Module Level 1] | |||
** Is investigating the spec of Motion Path | |||
* {{bug|1421507}} RESOLVED::FIXED Throttle animations on position:fixed element which is out-of-view | |||
** Reviewed bug 1421507 - Throttle animations on position:fixed element which is out-of-view | |||
* {{bug|1420928}} RESOLVED::FIXED stylo: animation-name and transition-property probably shouldn't be early properties. | |||
** Reviewed bug 1420928 - stylo: animation-name and transition-property probably shouldn't be early properties. | |||
* {{bug|1418806}} REOPENED:: Crash in core::result::unwrap_failed<T> | geckoservo::glue::Servo_AnimationCompose | |||
** Wrote a patch to dump the alignmend size for bug 1418806 | |||
===Cameron McCormack=== | ===Cameron McCormack=== | ||
* {{bug|1427419}} ASSIGNED:: convert inIDOMUtils to Web IDL | |||
** Uploaded new patches for bug 1427419 | |||
* {{bug|1427512}} ASSIGNED:: remove XPIDL interfaces for style sheets and style rules | |||
** Uploaded patches for bug 1427512 | |||
* {{bug|1428491}} NEW:: Make ServoStyleSet hold a reference to a document, not to a pres context. | |||
** Reviewed bug 1428491 | |||
* {{bug|1428276}} RESOLVED::FIXED Make ExpectedOwnerForChild deal with abs.pos. anon boxes | |||
** Reviewed bug 1428276 | |||
* {{bug|1428339}} NEW:: Make mapped attribute resolution not require a pres context. | |||
** Reviewed bug 1428339 | |||
* {{bug|1415781}} ASSIGNED:: Fix failure in layout/style/test/chrome/ relying on non-comformant Promise handling | |||
** Reviewed bug 1415781 | |||
* {{bug|1424607}} NEW:: Make sure invalidation handles ::slotted correctly. | |||
** Reviewed bug 1424607 | |||
* {{bug|1425755}} NEW:: Implement the missing selector-matching bits for ::slotted | |||
** Reviewed bug 1425755 | |||
===CJ Ku=== | ===CJ Ku=== | ||
* {{bug|1207734 }} Implement individual transform properties: the 'translate', 'scale', and 'rotate' properties | |||
** r? | |||
===Eric Tsai=== | ===Eric Tsai=== | ||
* Issue triage and diagnosis | |||
** Bottom of Chinese characters are cropped, even if it's web font https://webcompat.com/issues/10359 https://webcompat.com/issues/10477 https://webcompat.com/issues/10388 | |||
* Survey webcompat.com show similar bugs feature rework effort | |||
* Survey possible candidates for webcompat go faster system addon site patch | |||
===Jeremy Chen=== | ===Jeremy Chen=== | ||
* {{Bug|1283222}} - Unwanted line-break opportunities at out-of-flow frames [was: Commas wrapped to beginning of line with text-align: justify] | |||
** debug and clarify the unknowns (discussed with the reviewer during the y'allhands) | |||
===Louis Chang=== | ===Louis Chang=== | ||
* {{Bug|1428676}} - -moz-appearance meulist-button should behave like -webkit-appearance: menulist-button | |||
** filed a bug | |||
** WIP | |||
* {{Bug|1246836}} - -moz-appearance button should behave like -webkit-appearance: button | |||
** WIP | |||
** review mats patch | |||
===Ting-Yu Lin=== | ===Ting-Yu Lin=== | ||
* {{Bug|1404222}} - Support shape-outside: <image> | |||
** WIP | |||
===Tommy Kuo=== | ===Tommy Kuo=== | ||
* {{bug|1428276}} RESOLVED::FIXED Make ExpectedOwnerForChild deal with abs.pos. anon boxes | |||
** File bug and upload a patch to ask to review (bug 1428276) | |||
** Landed (bug 1428276) | |||
* {{bug|1428297}} NEW:: Get rid of using widget/view to show the dropdown menu when content-select is enabled. | |||
** File bug (Bug 1428297) | |||
** Uploaded the patch (bug 1428297) | |||
* {{bug|1425088}} NEW:: Change the dropdown frame of <select> to a top-level absolute frame | |||
** Make the dropdown frame be a top-level absolute frame behind a pref (bug 1425088) | |||
** Upload patches to ask to review (bug 1425088) | |||
* {{bug|1428701}} ASSIGNED:: Allow to toggle the dropdown frame when content-select is enabled | |||
** Filed a bug and WIP (bug 1428701) | |||
===Wei-Cheng Pan=== | ===Wei-Cheng Pan=== | ||
* {{bug|1427995}} NEW:: Repeatly evaling a string literal is 10x slower than Chrome | |||
** Filed bug 1427995 | |||
===Ya-Chieh Wu=== | ===Ya-Chieh Wu=== | ||
* {{bug|347335}} RESOLVED::INCOMPLETE css border-collapse bad rendering on huge tables | |||
** Close Bug 347335 | |||
* {{bug|1421887}} RESOLVED::FIXED [BC] |ResetBStartStart| might introduce weird borders if we don't get the row index right. | |||
** Land Bug 1421887 | |||
* {{bug|935914}} RESOLVED::DUPLICATE Collapsed borders are not drawn correctly in a table with rowspans | |||
** Close bug 935914 | |||
==Rule of Thumb== | ==Rule of Thumb== |
Latest revision as of 06:24, 9 January 2018
Weekly Announcement
- CultureAMP survey will close on Friday, January 12.
Weekly Sharing
Weekly Status
Astley Chen
- bug 1357966 NEW:: Don't report resource timing values for image, -moz-image-rect, font, and cursor from cross-origin stylesheets
- Bug 1357966 - sec-high and need work
Boris Chiou
- Blink issue 496518
- Checked the current implementation of motion path on Blink
- Motion Path Module Level 1
- Is investigating the spec of Motion Path
- bug 1421507 RESOLVED::FIXED Throttle animations on position:fixed element which is out-of-view
- Reviewed bug 1421507 - Throttle animations on position:fixed element which is out-of-view
- bug 1420928 RESOLVED::FIXED stylo: animation-name and transition-property probably shouldn't be early properties.
- Reviewed bug 1420928 - stylo: animation-name and transition-property probably shouldn't be early properties.
- bug 1418806 REOPENED:: Crash in core::result::unwrap_failed<T> | geckoservo::glue::Servo_AnimationCompose
- Wrote a patch to dump the alignmend size for bug 1418806
Cameron McCormack
- bug 1427419 ASSIGNED:: convert inIDOMUtils to Web IDL
- Uploaded new patches for bug 1427419
- bug 1427512 ASSIGNED:: remove XPIDL interfaces for style sheets and style rules
- Uploaded patches for bug 1427512
- bug 1428491 NEW:: Make ServoStyleSet hold a reference to a document, not to a pres context.
- Reviewed bug 1428491
- bug 1428276 RESOLVED::FIXED Make ExpectedOwnerForChild deal with abs.pos. anon boxes
- Reviewed bug 1428276
- bug 1428339 NEW:: Make mapped attribute resolution not require a pres context.
- Reviewed bug 1428339
- bug 1415781 ASSIGNED:: Fix failure in layout/style/test/chrome/ relying on non-comformant Promise handling
- Reviewed bug 1415781
- bug 1424607 NEW:: Make sure invalidation handles ::slotted correctly.
- Reviewed bug 1424607
- bug 1425755 NEW:: Implement the missing selector-matching bits for ::slotted
- Reviewed bug 1425755
CJ Ku
- bug 1207734 Implement individual transform properties: the 'translate', 'scale', and 'rotate' properties
- r?
Eric Tsai
- Issue triage and diagnosis
- Bottom of Chinese characters are cropped, even if it's web font https://webcompat.com/issues/10359 https://webcompat.com/issues/10477 https://webcompat.com/issues/10388
- Survey webcompat.com show similar bugs feature rework effort
- Survey possible candidates for webcompat go faster system addon site patch
Jeremy Chen
- bug 1283222 - Unwanted line-break opportunities at out-of-flow frames [was: Commas wrapped to beginning of line with text-align: justify]
- debug and clarify the unknowns (discussed with the reviewer during the y'allhands)
Louis Chang
- bug 1428676 - -moz-appearance meulist-button should behave like -webkit-appearance: menulist-button
- filed a bug
- WIP
- bug 1246836 - -moz-appearance button should behave like -webkit-appearance: button
- WIP
- review mats patch
Ting-Yu Lin
- bug 1404222 - Support shape-outside: <image>
- WIP
Tommy Kuo
- bug 1428276 RESOLVED::FIXED Make ExpectedOwnerForChild deal with abs.pos. anon boxes
- File bug and upload a patch to ask to review (bug 1428276)
- Landed (bug 1428276)
- bug 1428297 NEW:: Get rid of using widget/view to show the dropdown menu when content-select is enabled.
- File bug (Bug 1428297)
- Uploaded the patch (bug 1428297)
- bug 1425088 NEW:: Change the dropdown frame of <select> to a top-level absolute frame
- Make the dropdown frame be a top-level absolute frame behind a pref (bug 1425088)
- Upload patches to ask to review (bug 1425088)
- bug 1428701 ASSIGNED:: Allow to toggle the dropdown frame when content-select is enabled
- Filed a bug and WIP (bug 1428701)
Wei-Cheng Pan
- bug 1427995 NEW:: Repeatly evaling a string literal is 10x slower than Chrome
- Filed bug 1427995
Ya-Chieh Wu
- bug 347335 RESOLVED::INCOMPLETE css border-collapse bad rendering on huge tables
- Close Bug 347335
- bug 1421887 RESOLVED::FIXED [BC] |ResetBStartStart| might introduce weird borders if we don't get the row index right.
- Land Bug 1421887
- bug 935914 RESOLVED::DUPLICATE Collapsed borders are not drawn correctly in a table with rowspans
- Close bug 935914
Rule of Thumb
Status code:
- OOAD: In analysis or design phase.
- WIP: working on WIP patch.
- feedback(f): feedback a patch
- feedback?(f?): WIP is ready for feedback grant.
- feedback+(f+): Get positive feedback.
- feedback-(f-): Get negative feedback.
- review(r): review a patch.
- review?(r?): patch is under review
- review+(r+): get review+ and ready to check-in
- review-(r-): get negative review feedback. Fix problems and switch back to r? again.
- land: patch check in.
- debug: Figure out the root cause of a bug/ try to repro that bug, etc...
- file: file a bug
- PTO: take personal leave
- interview(I)
- study: Spend time on domain knowledge study, such as multimedia transport technology, compress format, real-time rendering.
- presentation
- Write a personal blog, which is relative to work, such as articles on MozTech, or what you experience in a work week.
- WIKI/ MDN article writing.
- If you write a technical blog or anything relative to work, it's welcome to share it on weekly report.
- Do a presentation of technical topics