TPE-Layout/2016-05-17

From MozillaWiki
Jump to: navigation, search

Weekly Report

Astley Chen

Offsite layout planning

  • Workweek rendering sessions planning.
  • 2016H2 planning.

CSS Masking - positioned mask

  • bug 1258286 - default value for mask-origin property is border-box but padding-box
    • WIP

Selector4 - unprefix :dir

  • bug 1270713 - Enable corresponding wpt tests for pseudo class :dir()
    • r+ - checkin-needed

Boris Chiou

  • bug 1244590 - Implement KeyframeEffect spacing modes
    • WIP | r-
  • bug 1272549 - Support paced spacing on transform, shape, and filter function
    • file

CJ Ku

  • bug 1269971 - Implement background-clip:text using a mask
    • r+
  • bug 1272973 - mask-repeat and mask-origin test case failed after enable MOZ_ENABLE_MASK_AS_SHORTHAND
    • filed
  • bug 1272970 - test_transitions_per_property.html failure after enable MOZ_ENABLE_MASK_AS_SHORTHAND
    • r?
  • bug 1269971 - Implement background-clip:text using a mask
    • landed
  • bug 1272978 - Split nsLayoutUtils::PaintFrame into two functions, one for build display list, one for paint display list
    • WIP

Jeremy Chen

WebCompat

  • bug 1267128 - -webkit-background-clip: text does not respect -webkit-text-stroke property
    • resolved as a duplicate of bug 1269971 which is landed by CJ

CSS

  • bug 1223880 - Implement initial-letter (from CSS Inline Layout Module Level 3)
    • WIP | OOAD
      • parsing & computing - 100%
      • rendering - study
      • tests - N/A
    • File related bugs:
      • bug 1273022 - Implement initial-letter-wrap (from CSS Inline Layout Module Level 3)
      • bug 1273021 - Implement initial-letter-align (from CSS Inline Layout Module Level 3)
      • bug 1273019 - [meta] Implement initial-letter related properties in CSS Inline Layout Module Level 3

Tingyu Lin

AccessibleCaret

Details & Summary

  • bug 1271549 - Remove preference "dom.details_element.enabled" for <details> and <summary>
    • file
  • bug 634004 - Make <details> and <summary> accessible
    • Test VoiceOver a bit, and needinfo for help.

Shape-outside

  • bug 1098939 - Implement CSS shape-outside property
    • Study 'float' layout, and add parser support.

Others (Provide information or comments, and triage bugs)

  • bug 1272831 - Add AutoRestore::Value() to retrieve the saved value
    • r?
  • bug 1272840 - Resurrect the block-and-line.html URL in nsFrameStateBits.h
    • land
  • bug 1273049 - Move the relevant info of frame bit flag from in block-and-line.html to nsFrameStateBits.h
    • file

Team

London workweek rendering sessions:

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)