AllMangasReader-dev / AMR

AllMangasReader developer branch
www.allmangasreader.com
Other
99 stars 56 forks source link

Chapter Display on Long Manhwa #80

Open bessx opened 11 years ago

bessx commented 11 years ago

@braiam Hey, I know you worked on the Batoto long page manga display, but I don't think we should break them up. We should put them back to back to make a long flowing image. If the user wants them broken up, they can always just click the "Page by Page mode" and AMR will handle it as it usually does. For the full (longpage) mode on Batoto I think it's better to either leave it as is or only change the black background/AMR bar.

braiam commented 11 years ago

Umm... then the right/left arrows hotkeys wouldn't work and that's the selling poing (check Easygoing site, everything there works just fine). Also, the images are not centered in the batoto default format and centering them breaks somehow the banner in the left.

And, just as kinda cheery in the top thing... why you pm'ing me from here? :astonished:

bessx commented 11 years ago

hahaha sorry!!! i was js here so i thought i'd bring it up. It's true that the left-right key functionality would be broken, but there's a button right on the top of the page that says "Want to see this chapter per page instead?". In that mode, AMR would work properly and it centers it properly. There's no reason to have AMR do the same exact thing on the longpage mode, is there? It would be better to only do a few changes in that case. (Such as centering and black background and amr bar)

And sorry for msging you on here, I was js playing with the new github design!

bessx commented 11 years ago

also, the reason it doesnt center properly is probably the little box on the top right. It's in a div that's float:right:

<div style="width:22%; text-align-left; float:right;">
<div style="width: 150px; padding: 5px; border-radius: 3px; background-color: #555555; margin-bottom: 10px; text-align: center; margin: 0 auto;">
<a href="?supress_webtoon=t" rel="nofollow" style="color: white; font-weight: bold;">Want to see this chapter per page instead?</a>
</div>
bessx commented 11 years ago

Actually I think it's the Width: 77% div right below that that's glitching the centering.