TPE-Layout/2016-12-06

From MozillaWiki
Jump to: navigation, search

Weekly Summary

Weekly Sharing

Weekly Status

Astley Chen

Boris Chiou

Cameron McCormack

CJ Ku

Farmer Tseng

Jeremy Chen

CSS Text Module Level 3

  • bug 1316482 - Whitespace around the segment break should be removed before segment break transformation
    • study | f- | OOAD

Layout

  • bug 1320239 - Use nscoord for some nsStyleCoord in style structs that only carry nscoord
    • land

Other

  • bug 1319383 - Do not check watchman extension if Mercurial version >= 3.8
    • file | r?

Shing Lyu

Tingyu Lin

layout

  • bug 1319672 - Improve document related to writing-mode in nsFloatManager
    • land
  • bug 1304598 - Drop "ns" prefix in file name to make it consistent with the class name
    • land
  • bug 1320660 - inline-block element is clearing floating elements differently in nightly 53.0a1 (2016-11-27) (64-bit) for OS X
    • Investigate the issue, and suggest fixing the site.
  • bug 1321056 - Preemptively fix unified bustage in layout/svg
    • review
  • bug 1321166 - Move DetailsFrame into mozilla namespace
    • file | land
  • bug 1321387 - Preemptively fix unified bustage in layout/generic
    • review
  • bug 1321495 - Preemptively fix unified bustage in layout/style
    • review

AccessibleCaret

  • bug 1156160 - Blur event closes AccessibleCaret but leaves visible selection on page
    • r+
    • Backout due to reftest failure on Fennec.
  • bug 1321491 - Tweak prefs related to enable AccessibleCaret
    • file | land
  • bug 1031362 - .pressure is always 0.5 on Pointer Events in Firefox Nightly desktop
    • feedback on one of the patches related to AccessibleCaret

Tommy Kuo

Team Affairs

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

Status code combination rule
Combine status code with "|". For example 

** OOAD|WIP

Which means you work on both OOAD and WIP in the last week.

land status code
If your patch is target for specific branch, instead of central, postfix branch version. For exp 

** land(1.4)
Retrieved from "https://wiki.mozilla.org/index.php?title=TPE-Layout/2016-12-06&oldid=1156443"