]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/plugins/contrib/album/discussion.mdwn
Merge branch 'ready/templatebody'
[git.ikiwiki.info.git] / doc / plugins / contrib / album / discussion.mdwn
index b7c9b009509f3e48e4f7ccd023a012e995c71332..23ff9017d366b3d407ce6788c9bf05856bbf91e6 100644 (file)
@@ -772,7 +772,12 @@ My wishlist for the plugin would include:
 >>>>
 >>>>> It can't *only* be an inline, because an inline wouldn't generate the
 >>>>> viewer pages, but I see what you mean. --s
 >>>>
 >>>>> It can't *only* be an inline, because an inline wouldn't generate the
 >>>>> viewer pages, but I see what you mean. --s
->>>> 
+>>>>>
+>>>>>> That's actually excellent as the inline is a very useful feature
+>>>>>> the way it works now. I started writing about this yesterday but
+>>>>>> got interrupted. My indexes of albums use the inline in it's current
+>>>>>> form. --k
+>>>>  
 >>>> This would make the viewers completely independent allowing for unique titles, captions and comments
 >>>> depending on context. Very useful when creating powerpoint like slideshows where you might need 
 >>>> different captions depending on the context. In your example wiki with photos from gigs this would allow 
 >>>> This would make the viewers completely independent allowing for unique titles, captions and comments
 >>>> depending on context. Very useful when creating powerpoint like slideshows where you might need 
 >>>> different captions depending on the context. In your example wiki with photos from gigs this would allow 
@@ -824,3 +829,61 @@ to `doc/style.css` should be enough? --[[smcv]]
 >
 >> I searched for `/* relevant to the index page */` and found it twice,
 >> so I stand by what I said :-) --s
 >
 >> I searched for `/* relevant to the index page */` and found it twice,
 >> so I stand by what I said :-) --s
+>>
+>>> And right you are, unsure how I missed that. My album branch is now rebased
+>>> on your album5 branch (with the two now useless commits removed).
+>>> --[[cbaines]]
+
+cbaines, would you mind publishing an album with more realistic pixel-sizes
+of images using your modified CSS? It's difficult to get an idea of how it
+will degrade under conditions like "image size > browser window" with
+images as small as the ones you used. You might find
+<http://git.pseudorandom.co.uk/smcv/ikiwiki-demos/ikialbum.git>
+(`git clone git://git.pseudorandom.co.uk/git/smcv/ikiwiki-demos/ikialbum.git`),
+or the same techniques, useful: it contains images with a realistic pixel
+count, but very very lossy JPEG compression, to keep the size in bytes low.
+
+> I have now created a large (images) example, you can find all the examples
+> here [1]. I have also built all the examples with the album5 branch, you can
+> find the results here [2].
+>
+> 1: <http://cbaines.net/projects/ikiwiki/album/dest/>
+> 2: <http://cbaines.net/projects/ikiwiki/album/dest-album5/>
+
+It's much, much easier to review changes if you use separate commits for
+cosmetic changes like "separate index CSS from viewer CSS" and "more
+consistent indentation", and functional changes like turning the prev/next
+links from absolutely-positioned to floating. I'd be happy to apply
+the cosmetic changes if they were in commits that were literally only
+cosmetic changes, with no functional effect.
+
+> I have now rewritten the CSS changes to get a smaller diff. The only big
+> functional change is from the previous patch is the max-width stuff to cope
+> better with large images.
+
+For the functional bits: I think I'd have used floating boxes instead of the
+absolutely-positioned boxes that are currently used if they provided the effect
+I wanted. I can't remember exactly why I didn't do that now, but
+it might have been because if the browser window shrinks below the image width,
+floats have weird behaviour (they push the actual image out of the way), or because
+I wanted the entire left/right margin of the image to be clickable to have
+a large click-target for scrolling through the album.
+
+If there's something specific that you think is wrong with the CSS in my
+branch, could you please explain it, and perhaps we can come up with something
+that matches both our requirements?
+
+--smcv
+
+> I don't think that something specific is wrong with CSS in the album5 branch,
+> but it does not display large [3], or small [4] images very well. It might be
+> possible to resolve the image size issues without changing from absolute
+> positioning, but I felt (for no particular reason) that I would do it using
+> floats.
+>
+> The clickable region on the margin seems the most likely reason to me to go
+> with absolute positioning, as an initial look at doing this with floats
+> suggests that it is non-trivial.
+>
+> 3: <http://cbaines.net/projects/ikiwiki/album/dest-album5/large-goldtype/album/3/>
+> 4: <http://cbaines.net/projects/ikiwiki/album/dest-album5/basic-blueview/album/ikiwiki_old/>