Skip to content
This repository was archived by the owner on Aug 2, 2024. It is now read-only.

Conversation

tiembo
Copy link
Contributor

@tiembo tiembo commented Jan 31, 2019

Changes include:

  • Remove the duplicate toolbar in the collapsing layout
  • Show plant name above watering details
  • Change text size of plant description

We will look into a different layout when we start implementing Material design.

Changes include:
- Remove the duplicate toolbar in the collapsing layout
- Show plant name above watering details
- Change text size of plant description

We will look into a different layout when we start implementing Material design.
@tiembo tiembo requested a review from nic0lette January 31, 2019 22:23
@tiembo tiembo added this to the 2019-01 milestone Jan 31, 2019
@tiembo tiembo added bug Something isn't working enhancement New feature or request labels Jan 31, 2019
@tiembo tiembo requested a review from ggfan January 31, 2019 22:24
@tiembo tiembo merged commit cde2c73 into master Jan 31, 2019
@tiembo tiembo deleted the ts/fix_toolbar branch January 31, 2019 23:49
@tiembo
Copy link
Contributor Author

tiembo commented Jan 31, 2019

Fixes #75

<TextView
android:id="@+id/plant_watering"
android:id="@+id/plant_name"
style="?android:attr/textAppearanceLarge"
Copy link
Contributor

Choose a reason for hiding this comment

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

Using android:textAppearance would be more suitable than using style as this attr only affects text display

@andhie andhie mentioned this pull request Feb 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants