search

FabianTerhorst / FastLayout

Generates a Java Object for your xml layout to reduce inflate time to zero
Apache License 2.0
443 stars 35 forks source link

Various bugs #2

Closed ivanfeli closed 8 years ago

ivanfeli commented 8 years ago

All these bugs were found with this layout: https://gist.github.com/drivfe/c8dcc6efb5dac448cbd5eebc9bcca85f

Version: 0.0.1-alpha13

To summarize:

  1. Supports paddingLeft/Top/Bottom/Right but it does not work if you set just padding="10dp" which is supposed to do all four of them. Looks like .setPadding() must always take 4 params.
  2. Does not support "sp" for textSize
  3. dp values does not work for layout_height and layout_width
  4. toStartOf and toEndOf not supported (I think paddingStart/End neither)
  5. Processor is not changing @+id/ to @id/ every time
FabianTerhorst commented 8 years ago

alpha16 is supporting @+id/, because android is also support that

FabianTerhorst commented 8 years ago

alpha 17 is supporting the start and end