Closed halfempty closed 8 years ago
OK, noted. Seems to function pretty well, though.
@ncalcott I've added some fields to the Edit screen of the Carousel page. When you get a chance, will you populate these with captions and links?
Sure, will do.
Can we add the 'News' page or some way to link to it into the associated portfolio dropdown somewhere? I may end up using this carousel to post recent work, in which case I'll want to just link to that category page as I won't have posted a portfolio yet....
@ncalcott Would it be better if that field was an open URL field, so that you could link to anything? If so, I will change it right now
I dunno if it'd be better, but if it's easier and will save some dev time it's not a major functionality issue for me - so, sure, go for it. Thanks!
@ncalcott Okay, fieldset updated
Hey all (@halfempty, @KyleETC, @gehalber)-
For the carousel, what do you guys think about having the left/right arrows disappear and are only visible on rollover?
Seeing as the site's about showing images, I'd prefer to keep them as uncluttered as possible. Obviously, if we do this, we'd have to make the rollover area quite a bit larger than just immediately over the arrows...
Also, wouldn't mind returning the close-window 'x' to the news box, so a viewer can make it disappear if they just want to see the image in the carousel. I don't really mind if the only way they can bring it back is to reload the page - chances are they don't need the info anymore, and the natural ux if they do would be to click on 'News', which will take them to the full archive...
Sorry, another minor thing - can we set the images to show up in random order? I know it's counter-intuitive (given that I have the ability to edit the image order on the backend), but I'd love the site to always feel fresh whenever someone visits, and I don't really have the patience to revisit the site frequently just to change the front page order...
@ncalcott Random order: Check!
@halfempty - I'm hoping to do an e-mail blast in the first week of September. I realize there's still a few things to get to, but any chance we could have the captions on the carousel show up and the arrows only show on hover before then? I'd also love to fix the updates module (issue #51) before that, if possible...
I added in the caption fields that hadn't been implemented yet and changed them to WSIYG editors in the Custom Fields plugin. Also, I modified the opacity of the backgrounds of the next/prev buttons. These are meant as temporary fixes, of course...
One issue with the way I've implemented this is that the captions are floating at the bottom of the window and not directly below the image itself, contrary to the original design. This isn't intentional - I just didn't have time to rejigger the CSS to get it to work as intended...
@ncalcott @gehalber I've done some additional caption jiggering:
Hey hey - looks good. Thing is, the images have disappeared in the carousel when viewing on safari in non-mobile view (the work in the reduced width css)...
@ncalcott Hmm. Working on my end. I've updated the version number, in case it's a cache issue. Can you confirm that it's still happening? Thanks!
Yep, working now. Thanks!
Question for @gehalber and @KyleETC --
In the mockup I'm seeing a couple of different treatments of the arrows. Which one should I be looking at?
Arrows as buttons:
Or arrows as cursors:
Arrows as buttons
@gehalber Okay, buttons updated! Closing this...
I've begun on this. But not finished.