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

Wide screen view improvements #2337

Closed
wants to merge 2 commits into from
Closed

Wide screen view improvements #2337

wants to merge 2 commits into from

Conversation

timocov
Copy link

@timocov timocov commented Aug 8, 2016

Hi guys,
this merge request refers to #2060 (and similar issues).
Below you can find comparison screenshots.

Messages before:
Messages before

Messages after:
Messages after

Group/Bot/Channel info before:
Group/Bot/Channel info before

Group/Bot/Channel info after:
Group/Bot/Channel info after

Signed-off-by: Eugene Timokhov <timocov@gmail.com> (github: timocov)
Signed-off-by: Eugene Timokhov <timocov@gmail.com> (github: timocov)
@timocov
Copy link
Author

timocov commented Aug 9, 2016

Max size of photo's dimension is 420px, so we can use it space for photos.

Photo with caption before:
before

Photo with caption after:
after

@auchri
Copy link
Contributor

auchri commented Aug 9, 2016

Is this still WIP?

@auchri auchri added WIP and removed needs review labels Aug 9, 2016
@timocov
Copy link
Author

timocov commented Aug 9, 2016

Yeah, sorry about this. I just forget about some things. I reopen pr when it will be ready

@timocov timocov closed this Aug 9, 2016
@timocov
Copy link
Author

timocov commented Aug 9, 2016

I cannot reopen this pr :(
I will create one more with this changes.
Sorry again

@github-actions
Copy link

github-actions bot commented Mar 4, 2021

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants