Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tiny vertical images in reader post detail #3068

Closed
roundhill opened this issue Aug 13, 2015 · 3 comments
Closed

Tiny vertical images in reader post detail #3068

roundhill opened this issue Aug 13, 2015 · 3 comments
Assignees

Comments

@roundhill
Copy link
Contributor

I created a gallery post in the wp.com editor, and noticed that the android reader displayed the vertical images very small:

https://cloudup.com/c3EJ55-0WkT

http://danroundhill.com/2015/08/13/aberdeens-star-wars-shop/

Assigned to you @nbradbury :)

@oldcwj
Copy link

oldcwj commented Aug 29, 2015

Nexus 7
screenshot_2015-08-29-15-52-27

ReaderPostRenderer.java->replaceImageTag(final String imageTag, final String imageUrl)
There are four cases(isFullSize, isMidSize, hasWidth, other), only (isFullSize) is true which use screen width, (isMidSize,hasWidth and other) use self width, so that is the problem, screen width and image`s width are not have relationship(like scale),so (isMidSize, hasWidth) are too small.

@rachelmcr
Copy link
Member

I confirmed that this is still an issue in the latest beta (5.0-rc-2). It would be great to improve this detail view for galleries as part of #1719 (along with adding support in the list view).

@hypest
Copy link
Contributor

hypest commented Jun 22, 2016

Fixed with #4222 . Closing.

@hypest hypest closed this as completed Jun 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants