687
edits
TPE-Layout/2017-09-12: Difference between revisions
Jump to navigation
Jump to search
→Weekly Status: 2017W37
Astleychen (talk | contribs) (→Weekly Announcement: 2017W37) |
Astleychen (talk | contribs) (→Weekly Status: 2017W37) |
||
| Line 15: | Line 15: | ||
==Weekly Status== | ==Weekly Status== | ||
===Astley Chen=== | ===Astley Chen=== | ||
* {{bug|752790}} NEW:: Input padding covers text when the padding is greater than (TextHeight - InputHeight)/2 | |||
** Bug 752790 - long standing webcompat issue needs some love. | |||
===Boris Chiou=== | ===Boris Chiou=== | ||
* {{bug|1397146}} NEW:: The length part of calc() should not be rounded to nscoord (integer) for computed transform | |||
** Filed bug 1397146 - The length part of calc() should not be rounded to nscoord (integer) for computed transform | |||
* {{bug|1396692}} RESOLVED::FIXED stylo: unit of absolute length needs to be preserved in calc() | |||
** Landed Bug 1396692 - stylo: unit of absolute length needs to be preserved in calc() | |||
** Wrote patches for bug 1396692 - stylo: unit of absolute length needs to be preserved in calc() | |||
* {{bug|1392161}} ASSIGNED:: stylo: Length values should not be rounded to Au for computed transform | |||
** Wrote patches to replace computed::Length with CSSPixelLength for bug 1392161 - stylo: Length values should not be rounded to Au for computed transform | |||
===Cameron McCormack=== | ===Cameron McCormack=== | ||
* Reviewed https://github.com/servo/servo/pull/18400 | |||
* Reviewed https://github.com/servo/servo/pull/18404 | |||
* {{bug|1393861}} RESOLVED::FIXED stylo: switching Slack channels sometimes requires multiple clicks | |||
** Reviewed bug 1393861 | |||
* {{bug|1395227}} RESOLVED::FIXED stylo: stop the cascade if only reset properties change but no children explicitly inherited | |||
** Reviewed bug 1395227 | |||
* {{bug|1342951}} NEW:: Expensive sync reflows when holding the down key on google presentations | |||
** Reviewed rest of bug 1342951 | |||
* {{bug|1397238}} ASSIGNED:: Crash in InvalidArrayIndex_CRASH | gfxFontFamily::FindAllFontsForStyle | |||
** Reviewed bug 1397238 | |||
* {{bug|1367635}} NEW:: stylo: consider caching style structs in the rule tree | |||
** Refining bug 1367635 patch to use less memory | |||
* {{bug|1387958}} RESOLVED::FIXED stylo: Measure the entire stylist during memory reporting | |||
** Reviewed bug 1387958 | |||
* {{bug|1387410}} RESOLVED::FIXED make grid-row-gap and grid-column-gap no longer be reset by the grid shorthand | |||
** Reviewed bug 1387410 | |||
* {{bug|1397579}} RESOLVED::FIXED Report stylesheet memory usage under "layout/style-sheets" | |||
** Reviewed bug 1397579 | |||
===CJ Ku=== | ===CJ Ku=== | ||
* {{bug|1396025}} RESOLVED::FIXED Write a comment to explain the difference between eBBoxIncludeFill and eBBoxIncludeFillGeometry | |||
** Bug 1396025 land | |||
* {{bug|1396776}} RESOLVED::FIXED Move mCanvasTM from the derived class of nsSVGViewportFrame to nsSVGViewportFrame | |||
** Bug 1396776 land | |||
* {{bug|1381153}} ASSIGNED:: Calling GetEffectSet during display list building is not very cache friendly | |||
** Bug 1381153 upload patch | |||
* {{bug|1396489}} RESOLVED::FIXED Let gfxContext::GetClipExtents be able to return clipped rect in either user space or device space | |||
** Bug 1396489 landed | |||
* {{bug|1294966}} RESOLVED::WONTFIX SVG getBBox should provide useful information for zero-width/height shapes | |||
** Bug 1294966 upload a patch | |||
===Eric Tsai=== | ===Eric Tsai=== | ||
| Line 27: | Line 64: | ||
===Jeremy Chen=== | ===Jeremy Chen=== | ||
* Filed a issue about the perma failures on a specific mac build slave [servo #18374] | |||
* {{bug|1391341}} NEW:: stylo: Font size inherits differently than in Gecko | |||
** Triaged and un-assigned myself [Bug 1391341] | |||
* {{bug|1296577}} RESOLVED::DUPLICATE Give nscoord_MAX a better definition to avoid potential integer overflowing | |||
** Replied ni and resolved the bug [Bug 1296577] | |||
* {{bug|1398011}} NEW:: [META] Implement CSS Transform Module Level 2 | |||
** Filed Bug 1398011 | |||
* {{bug|1394284}} ASSIGNED:: stylo: animate singular matrix as discrete | |||
** Investigated a bit, NI for more context [Bug 1394284] | |||
** Investigated, came up with a rough plan, NI Brian for feedback [Bug 1394284] | |||
** Reported implementation progress, need to further refine the design plan [Bug 1394284] | |||
** Uploaded a patch and asked for 1st round of review [Bug 1394284] | |||
** Uploaded a wip [Bug 1394284] | |||
* {{bug|1394994}} RESOLVED::FIXED Stylo: Keyframe rules missing line / column info | |||
** Landed [Bug 1394994] | |||
===Louis Chang=== | ===Louis Chang=== | ||
===Ting-Yu Lin=== | ===Ting-Yu Lin=== | ||
* Translate "Duplicate Tab" as "複製分頁" for zh_TW in https://pontoon.mozilla.org/zh-TW/firefox/browser/chrome/browser/browser.dtd/?string=171582 | |||
* {{bug|1396700}} RESOLVED::FIXED stylo: Minor clean up for ServoStyleSet | |||
** Filed and landed bug 1396700. | |||
* {{bug|1340966}} NEW:: Add assertion to nsReflowStatus::SetNextInFlowNeedsReflow() | |||
** Uploaded a patch for bug 1340966. | |||
===Tommy Kuo=== | ===Tommy Kuo=== | ||
* {{bug|1229979}} ASSIGNED:: Multi-color shadow shouldn't be allowed when the color of text is different to the color of decoration line. | |||
** Debug for the failure of windows and android (bug 1229979) | |||
** Landed (bug 1229979) | |||
** Read the spec resolution and update the issue (bug 1229979) | |||
** Upload patches and ask to review (bug 1229979) | |||
** WIP and ran a try (bug 1229979) | |||
* {{bug|1380590}} RESOLVED::FIXED stylo: the fill color should be transparent when `svg.context-properties.content.enabled=false` and no fallback color given | |||
** Got r+ and wait to land (bug 1380590) | |||
** Landed (bug 1380590) | |||
===Ya-Chieh Wu=== | ===Ya-Chieh Wu=== | ||
* Bug895096 patch uploaded | |||
* Debug Bug1381153 | |||
* {{bug|1397119}} NEW:: Move GetBStartBCBorderWidth, GetBEndBCBorderWidth and any similar methods' return type to BCPixelSize rather than nscoord | |||
** File Bug 1397119 | |||
==Rule of Thumb== | ==Rule of Thumb== | ||