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

Avoid qrcode getting cropped #1055

Merged
merged 1 commit into from
Apr 10, 2019

Conversation

rponline
Copy link

@rponline rponline commented Apr 9, 2019

increase image width for better scanning

remove height to avoid white padding over and under qrcode
(modal box uses the height for content size, but css resize gets ignored)

Before After
Screen Shot 2019-04-09 at 15 29 27 Screen Shot 2019-04-09 at 15 28 46

increase image width for better scanning

remove height to avoid white padding over and under qrcode
(modal box uses the height for content size, but css resize gets ignored)
@codecov
Copy link

codecov bot commented Apr 9, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1055   +/-   ##
======================================
  Coverage      60%     60%           
======================================
  Files           4       4           
  Lines          60      60           
======================================
  Hits           36      36           
  Misses         24      24

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 989119f...e719f5d. Read the comment docs.

@skjnldsv
Copy link
Member

Thanks for all you've been doing lately @rponline 🤗

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working design Related to the design labels Apr 10, 2019
@skjnldsv skjnldsv added this to the next milestone Apr 10, 2019
@skjnldsv skjnldsv merged commit e573b41 into nextcloud:master Apr 10, 2019
@skjnldsv skjnldsv modified the milestones: next, 3.1.1 Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working design Related to the design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants