darktable icon indicating copy to clipboard operation
darktable copied to clipboard

Modernize WebUI Gallery export

Open marceloexc opened this issue 1 year ago • 7 comments

old webui gallery exporter did not work very well with mobile screens (everything appeared too small and not reactive)

kept the same hanatos style, but started using HTML standard elements (<header>, <footer>, <figure>, instead of everything being <div> <div> <div>...)

switched from xhtml to the html5 (no self closing tags for elements)

marceloexc avatar Aug 30 '24 02:08 marceloexc

Would be nice to have templates with html and css files in a folder under config/darktable to give users a way to create their own galleries. Like with the watermarks.

zisoft avatar Aug 30 '24 08:08 zisoft

let me know what you think about the new padding vertically

marceloexc avatar Aug 31 '24 16:08 marceloexc

Would be nice to have templates with html and css files in a folder under config/darktable to give users a way to create their own galleries. Like with the watermarks.

i agree. i think maybe having a templating engine would be a bit too much for the html but i definitely see having the css file available to change so that people can make their own colors, have a different grid layout etc.

unfortunately i dont know much about c so i can't volunteer for it. but it is a good idea

marceloexc avatar Aug 31 '24 16:08 marceloexc

I don't have more spacing for the gallery title:

image

TurboGit avatar Sep 01 '24 05:09 TurboGit

@marceloexc : Is that PR still in your scope? Would be nice to fix the CSS issue.

TurboGit avatar Oct 12 '24 10:10 TurboGit

sorry about the long delay, ill get a working commit by this week

marceloexc avatar Oct 15 '24 15:10 marceloexc

sorry about the long delay, ill get a working commit by this week

Don't be sorry, all this is done on our free time!

TurboGit avatar Oct 15 '24 15:10 TurboGit

came back to this, i have the correct gap on my machine? when you do inspect element on the browser, do you see the

element having padding-inline: 20px; padding-block: 40px; applied instead of just padding: 20px? this online gallery works on the devices i tested - could this be a matter of browser cache?

marceloexc avatar Nov 09 '24 23:11 marceloexc

this online gallery works on the devices i tested - could this be a matter of browser cache?

To me this example doesn't work. In your title "nov9_dt_web_test" you don't have letter having low jamb (don't know the right word, in French - Jambage bas) like g, y, p or q and those letters gets cut. Just try and you'll see.

TurboGit avatar Nov 11 '24 18:11 TurboGit

@marceloexc : Any new about this?

TurboGit avatar Jan 22 '25 18:01 TurboGit

Should we look at #16205 here too? Using CSSbox may satisfy the modernization needs.

wpferguson avatar Feb 03 '25 17:02 wpferguson

Moving this to 5.4 for now.

TurboGit avatar May 16 '25 06:05 TurboGit