[ home ] [ pony / townhall / rp / canterlot / rules ] [ arch ]

/canterlot/ - Canterlot

Site related staff board
Password (For file deletion.)

[Return][Go to bottom]


File: 1540000339350.jpg (43.9 KB, 640x960, 2:3, dog-face-found-in-nerds-ha….jpg) ImgOps Exif Google

's feature request: Option to downsize thumbnails of non-OP images.  I don't think this can easily be implemented client-side with just CSS, but JavaScript+CSS can probably get the job done.  Or it could be done server-side.  I could also take a stab at doing this if you give me read-only access to the source code and let me submit a pull request or something.

Pic unrelated.


I support this.


Personally, I like the size as-is. This might be due to having a larger screen, however.
If you do put it in, please make sure it's optional.




I prefer them the size they are.


File: 1540342001582.jpg (154.3 KB, 643x720, 643:720, 1520046934574.jpg) ImgOps Exif Google

Yeah, the idea is to make it **optional** to make the thumbnails smaller.  If the OP feature request gets implemented, you'd still need to affirmatively opt-in to the smaller thumbnails.


I'm trying to get the right css rule here, but I don't know what you and Iara are really expecting from this. Just, 50% smaller? Or capped at 100px? Should youtube embed images shrink? I dunno, this is what I have right now

.post.reply .post-image:not(.embedimg) { max-width: 100px; height: auto !important; }

If you want to refine the rule, be my guest. I already have worked out a pretty simple system for making dynamic css changes with javascript, so that part is easy once I have a rule in which to toggle on and off. I just don't want to bother putting this in place unless I know it's what is expected, because "downsize thumbnails" is not actually a usable specification.


I think the idea was to make it be like 4chan: constrain the height and width to each be less than or equal to 125px, while keeping the aspect ratio the same.  But since it seems that the images are currently up to 230px tall/wide, making them 50% smaller (so up to 115px tall/wide, instead of 125px) would also work.

It seems to me that the CSS rule you posted above would work for wide images but not tall images.  But I guess if you label each image with a CSS class indicating whether it's landscape or portrait, you can have one rule for tall images and another for wide images.


I kinda started putting this off because I stopped seeing iara around. Is there a reason to hurry? Like I'd do it if it'd make her post more or something.


File: 1542573623953.png (4.63 KB, 230x56, 115:28, 20180110181540.png) ImgOps Google


Oh cool. I'll try to get it done in the next few days ^_^


>size shaming


I still support this and wait for it.

I love big messages with tiny images.


Sorry I'm like totally failing to get this done. Things have been very busy and stressful for me recently. There's kinda a few things I want to handle at the same time, probably at a new panel in the options for making these kinds of small tweaks such as the image size, someone asked about line spacing, etc. Can't make promises, but it's basically on my mind every day and I just need a block of free time to do it.


A UI would be nice to have, but if you just implement the functionality and tell us what variable in localStorage controls it, then we can change the value manually in the console debugger.


File: 1543287221435.png (367.2 KB, 1024x1280, 4:5, For lil ol me¿ You shouldn….png) ImgOps Google

Believe in the me who believes in you.

Can't say anything more than that. I know nothing about coding!


File: 1543539541969.png (150.08 KB, 507x454, 507:454, pinkyhoof.png) ImgOps Google


if Thorax does as you ask, will you hang out here more often?


>Sorry I'm like totally failing to get this done. Things have been very busy and stressful for me recently.
If you're still busy, I can take a look at this during this weekend and submit a pull request.  (I have experience with PHP, JavaScript, and HTML/CSS, so I should be able to do this fairly cleanly.)

I made a public key:
ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAACAQDRT/u0DVemX1xoROGOMiqPWBqpfdcKawe9TjJfIo94JqEGT3ioh8SCOASALOwR1c0fdJvU0gFVLXtISed3e9CyjFGbM10V5Jm8yRfgmj+oSSeU5yV78kSNYvgkJIZ9F8lTXpz9wMKSmfBiSYpza7uMfYDX8eWlKhfM21i1j5RCK28/b49WhB3ioPGyg3dOv8Nl2WNBjjeqmY+YxKonxLKNszBmE6MMWj2lYw9fzGwAxHo3Frzo+aMeHxFx0gKVkS9Frxc8NWdvQMHI89QSj6i7AYmgbY2BB0xuICoUzVVZHKcCBdtdoanta0dT+ByYDV4LFt0mANBIAXvCyoYEjMstSSWy4EWR5VAp7kNU7/0ZY26JiUbfIeqAX0+cyqReNcb2LmM4Is66Sj0IIvUW3unxLkGA5vcBekdJf/PYwWmm37t+v63RwaVIBQm1Y+wdK5N2T3ZP2b0K98aEb2Zh+QRWrmUD28HADtKFMhjOvbdkJ93yTuFPfbWrs/9EVB3S+i5XuilV6aZ7HR7n2Umrx0H/p7H5J96wqy9jMx/gHdIEyZ2resJRiuwhXXfOHzS8JQDD/T9IE85szl+g2txE4eWubpbQYUFV79wVmevFfdm1BnLli9hIEfvCCjQq891Cqx8kPTR4AtQyfWzJLbAoFm41dcP21+CqMtGbaAMq9xLi0Q== ChainWall


Yeah sure. I hate big thumbnails tho.


This is done, finally. It wasn't really that hard to do, but I had to shift a few things around to actually get the time, energy, and will to align to make it happen.

Thanks for the offer, it kinda helped motivate me to carve out the time to do this so you wouldn't have to go in and do it for me, lol. It's nice of you to offer, but I wanted to refactor some things to clean things up a bit. If you want access to the code still, just for funsies or whatever, I don't mind that. Let me know.


P.S. You probably already know this (from IP address if nothing else) but following post was me:
>Arigatou, Thorax-senpai!


I don't spy on IPs like that often. I knew it was you from your mannerisms ^_^

[Return] [Go to top]
[ home ] [ pony / townhall / rp / canterlot / rules ] [ arch ]