hicetnunc icon indicating copy to clipboard operation
hicetnunc copied to clipboard

Objkt display height

Open kylegrover opened this issue 3 years ago • 10 comments

increases objkt display page height for images and videos, shrinks & evens the margin around objkt-display container, remove box-shadow around .info, and fix broken galleries

kylegrover avatar Jun 18 '21 04:06 kylegrover

image

kylegrover avatar Jun 18 '21 05:06 kylegrover

Hey! Yesterday I've tried to raise that to 100vh and showed it to Rafael and he wasn't a fan. I managed to "push it" so it goes from 60vh to 70vh but check with him.

andrevenancio avatar Jun 18 '21 10:06 andrevenancio

Hey! Yesterday I've tried to raise that to 100vh and showed it to Rafael and he wasn't a fan. I managed to "push it" so it goes from 60vh to 70vh but check with him.

I think this kind of decision should be based on some community vote or something like that, not on someone's personal approval, most of the artists I know disapprove of the change in size of the objkts (which used to be bigger)

llumidev avatar Jun 18 '21 12:06 llumidev

Hey! Yesterday I've tried to raise that to 100vh and showed it to Rafael and he wasn't a fan. I managed to "push it" so it goes from 60vh to 70vh but check with him.

I think this kind of decision should be based on some community vote or something like that, not on someone's personal approval, most of the artists I know disapprove of the change in size of the objkts (which used to be bigger)

i couldn't agree more.

aevin-io avatar Jun 18 '21 14:06 aevin-io

@llumidev participation > vote

crzypatchwork avatar Jun 18 '21 17:06 crzypatchwork

I think there's some healthy balance of all of this stuff. Participation is a must or nothing happens, but we can sometimes use voting to guide our changes so we can breeze past these kinds of discussions. Also think this should be a healthy height balance, I used calc to set it so the collect button is always just barely in view, it ends up being pretty close to 70vh I think

kylegrover avatar Jun 18 '21 19:06 kylegrover

One minor change should maybe be made: I think I hardcoded #222 as the border color, should maybe be transparent gray or set by a css var so it's lighter on light mode. Sometimes I forget there are people out there not using dark mode 😎. Also I only have a couple of devices so if anyone can test on Mac, Android, anything else I might forget that's always beneficial for these changes.

kylegrover avatar Jun 18 '21 19:06 kylegrover

@andrevenancio could you try to talk to Rafael? To at least reach 90vh, it was much better before, I work with interactive pieces and this smaller size is so bad (I'm not the only one who thinks so), not only for interactive works but all the other types. Pleaseeee🙏

Karl-010 avatar Jun 18 '21 20:06 Karl-010

I added a proposal in basedao to set hicetnunc2000/hicetnunc/pull/908 to merge so we can try voting on this. This will be non-binding since we haven't declared that as a mechanism, but I think it's worth an experiment.

Anybody have recommendations for creating a public demo? Just deploy to Netlify / other static hosting? Would be nice to allow non-developers to try out PRs and vote / give feedback.

kylegrover avatar Jun 18 '21 21:06 kylegrover

@andrevenancio could you try to talk to Rafael? To at least reach 90vh, it was much better before, I work with interactive pieces and this smaller size is so bad (I'm not the only one who thinks so), not only for interactive works but all the other types. Pleaseeee🙏

same, my interactive pieces are looking really bad right now.

for example: https://www.hicetnunc.xyz/objkt/141104

the iframe was around 600px height when I mint this and it was twice as big because its original size is 300x300 and it uses letterbox integer scale.

bilgekaan avatar Jun 20 '21 07:06 bilgekaan