Skip to content
This repository was archived by the owner on Dec 8, 2022. It is now read-only.

moved last update to the top, changed look of load more comments button#474

Merged
AndyScherzinger merged 1 commit into
AndlyticsProject:devfrom
AndyScherzinger:dev
Jun 16, 2013
Merged

moved last update to the top, changed look of load more comments button#474
AndyScherzinger merged 1 commit into
AndlyticsProject:devfrom
AndyScherzinger:dev

Conversation

@AndyScherzinger

Copy link
Copy Markdown
Member

No description provided.

AndyScherzinger added a commit that referenced this pull request Jun 16, 2013
moved last update to the top, changed look of load more comments button
@AndyScherzinger
AndyScherzinger merged commit f291264 into AndlyticsProject:dev Jun 16, 2013
@AndyScherzinger

Copy link
Copy Markdown
Member Author

@nelenkov @willlunniss can you check if this works fine when having more than one app in an account (since I moved a spaced of the main_list_item from top to bottom to save some space on the top of the main screen).

Hope the changes are fine with you guys, I felt the last update should be on the top of the screen rather than at the bottom. :)

@willlunniss

Copy link
Copy Markdown
Contributor

I'm out of the country for work for the next week, so will have even less change to help out until I return, sorry.

@nelenkov

Copy link
Copy Markdown
Contributor

Looks OK to me. I almost never looked at the update date till now, I guess it makes sense for it to be on top. The load comments button looks nice too.

@AndyScherzinger

Copy link
Copy Markdown
Member Author

Is the space between the apps Ok on the main screen?

@nelenkov

Copy link
Copy Markdown
Contributor

Looks OK to me, maybe less than before. Do you want a screenshot?

@AndyScherzinger

Copy link
Copy Markdown
Member Author

That would be great! :-)

@nelenkov

Copy link
Copy Markdown
Contributor

Here's how it looks on an N4:

andlytics spacing

@AndyScherzinger

Copy link
Copy Markdown
Member Author

Thanks! I guess I should then add another 5dp to the spacer

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants