internetarchive / openlibrary

One webpage for every book ever published!
https://openlibrary.org
GNU Affero General Public License v3.0
5.26k stars 1.4k forks source link

Make changes in the editions component for tablets #2006

Open koderjoker opened 5 years ago

koderjoker commented 5 years ago

Page: http://openlibrary.org/works/OL15203543W/Meistersinger_von_N%C3%BCrnberg

Is your feature request related to a problem? Please describe.

Continuation of #1339. For screen widths <=789px and >500px the read ebook button is far too wide, further changes in the editions component are required. 0 0 0 0_8080_works_OL15203543W_Meistersinger_von_N%C3%BCrnberg(iPad)

Proposal & Constraints

Some previous discussions to start off from:


Originally posted by @tfmorris in #1899(comment)

Did reducing with width of the "Read" button get lost? I commented on it a few weeks ago and @koderjoker said:

For starters, the width of the 'read eBook button' definitely needs to be reduced. We then could have the 'daisy and download links' on one line and the 'library and purchasing links' on another, but the read ebook button would still have space left over on the side...

which sounded like agreement that it was too wide. Paradoxically, it's fine when the screen is wide, but below ~770 pix, it jumps to full width.


Originally posted by @koderjoker in #1899(comment)

Shifting download links to a dropdown (to overcome problem of whitespace and make download links bigger for mobile) Rough visualisation download button


Originally posted by @koderjoker in #1899(comment)

For tablet screen shot 2019-03-01 at 10 31 58 pm

Unfortunately, the tablet version still needs changes :p

For starters, the width of the 'read eBook button' definitely needs to be reduced. We then could have the 'daisy and download links' on one line and the 'library and purchasing links' on another, but the read ebook button would still have space left over on the side...

We could overcome that with a future download button though. The 'read ebook and download button' on one line, and 'library links and purchase links' on another.


However, the above would require deciding on a design for a dropdown.

jdlrobson commented 4 years ago

Maybe we want to expand this issue to be a redesign given the comments by @BrittanyBunk in https://github.com/internetarchive/openlibrary/issues/679#issuecomment-568232398

SaravgiYash commented 3 years ago

@jdlrobson I would like to work on this issue. Currently only Download for print-disabled is present and in another issue, I have added Worldcat link. So in this issue, I just have to reduce the Borrow button width?

My suggestion: We can keep the desktop design till width: 568px this will solve the issue. image

jdlrobson commented 3 years ago

This is mostly fixed.

Help fixing the size of the buttons on https://openlibrary.org/works/OL15203543W/Meistersinger_von_N%C3%BCrnberg here to be the width of the label is all that's needed to resolve this now:

Screen Shot 2021-01-13 at 6 04 34 PM

Help fixing that appreciated! Feel free!

The buttons in the screenshot below are fine and working as expected

image

github-actions[bot] commented 10 months ago

Assignees removed automatically after 14 days.