Open TheAssassin opened 6 years ago
+1 :D
Hey guys, I saw that issue and it looked pretty interesting to me, so I just threw together a mockup. How does it look, what do y'all think?
Main problem is that it is quite a bit of text... I also dont really like how the arrows are floating above the text, I'd probably disable them and only leave the little circles as navigators.
Feel free to create a new branch so we can comment on the code directly. Please use a .container
for the carousel text to align the left and right boundaries with the other ones. I'd also suggest to hide the arrows by default and show the on hover. Additionally you can then also add a gradient background to the arrow container to add some more contrast to it.
Instead of using the arrow background, you can also reduce the width of the text container further.
I thought I would wait with that until you publish your CSS rewrite @TobiGr. Or will that be unaffected?
I thought I would wait with that until you publish your CSS rewrite @TobiGr. Or will that be unaffected?
The CSS refactor is mostly about better class structure and CSS optimizations. This will be unaffected.
I though about the shadow again. I think we should not use it to keep a flat design.
good, okay. I have some other things on my plate, but I will implement a proper version soon and then we can have a look at it
A friendly user showed up on IRC recently to request some support, and liked what we did. He donated a few dollars, and sent us the following statement:
Before you ask: yes, he is really a reverend.
I think that review best fits on our donation page, not the homepage. @TobiGr can you suggest a place where to put it, please?
P.S.: Thanks for the kind words, Reverend Ben.