Skip to content

Move SummaryBottomSheet above RecenterBtn in navigation_view_layout.xml#1616

Merged
danesfeder merged 1 commit into
masterfrom
dan-update-xml
Dec 10, 2018
Merged

Move SummaryBottomSheet above RecenterBtn in navigation_view_layout.xml#1616
danesfeder merged 1 commit into
masterfrom
dan-update-xml

Conversation

@danesfeder
Copy link
Copy Markdown
Contributor

I'm not sure why this reproducible consistently, but I think #1614 and #1424 (comment) are valid.

It seems this error can occur when a view in the XML is trying to anchor to another view that hasn't been declared yet in the XML. In our case, the RecenterBtn was trying to anchor to the SummaryBottomSheet directly below it. See https://stackoverflow.com/a/11669528/8412108

@danesfeder danesfeder added bug Defect to be fixed. ✓ ready for review labels Dec 7, 2018
@danesfeder danesfeder self-assigned this Dec 7, 2018
@danesfeder danesfeder added this to the v0.25.0 milestone Dec 7, 2018
@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 7, 2018

Codecov Report

Merging #1616 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1616   +/-   ##
=========================================
  Coverage     22.76%   22.76%           
  Complexity      704      704           
=========================================
  Files           189      189           
  Lines          8244     8244           
  Branches        609      609           
=========================================
  Hits           1877     1877           
  Misses         6184     6184           
  Partials        183      183

Copy link
Copy Markdown
Contributor

@Guardiola31337 Guardiola31337 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me @danesfeder

@danesfeder danesfeder merged commit 9e932ac into master Dec 10, 2018
@danesfeder danesfeder deleted the dan-update-xml branch December 10, 2018 13:32
@danesfeder danesfeder mentioned this pull request Dec 16, 2018
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Defect to be fixed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants