GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||
GeezerVison is showing github.com — Display image name on download buttons and improve the buttons' formatting by WasabiFan · Pull Request #340 · ev3dev/ev3dev.github.io Conversation
|
This seems a very odd feature for an otherwise static website. Let's explore other possibilities. For example, could we get away with removing the cache? GitHub's rate limit is 60 requests per hour, so I have a hard time imagining that someone would hit this limit. Or we could setup a relay server as suggested here.
When you opened issue #340, I was envisioning having the file name on the button itself. Something like this (except with better alignment and a more narrow font for the file name): |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I don't think we can remove it entirely, but I think it would be fine to remove it or dramatically lower the timeout for the downloads page specifically. I'd like to avoid setting up our own infrastructure unless absolutely necessary.
I tried that, but I couldn't figure out how to make that happen within an |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I would rather do this than a reload button.
This says in HTML 5, you can put anything you like inside of an |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I've taken the time to come back to this PR and get it finished up. I removed the "refresh" button in favor of a decreased cache time and reformatted the download buttons to work well with an inlined file name. Here's how it looks (sorry for the huge screenshot 😉): I can't persuade our publish script to function properly right now so we'll have to settle for the screenshot. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I think the white background of the size badge makes the size look too "important". How about using a different shade of orange so that it is less prominent?
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I was able to find a tweak of that design which is a bit less bold: Personally, I think the darker color is distracting in its own right as it departs from the design we use on the rest of the site. Playing around in Illustrator, I came up with something I like, however it'd be a bit of work to build it in CSS: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I figured out how to do that multipart button, if we want it: http://www.bootply.com/v2c4s4X2Xe |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
bootply link is not working for me |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
Haha, looks like I broke Bootply... they're completely down right now 😁 That mock-up is just a technical demo of the alignment; the styling isn't polished by any means. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Hmm... the bootply demo interacts as if it is two separate buttons. The mouseover affects the top and bottom halves separately. That doesn't make for a very good UX. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
That's just some styling work that I didn't bother with for demonstration purposes; my goal was to make sure that I could handle the alignment of the containers, because mouseover effects are known to work. If I actually implement this control I'll need to style the button accordingly.. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I like it if you can make it work. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I have the basis of this done. I next need to get my styles working for dynamically-chosen button sizes so that the smaller ones for RasPi and the Beagles are handled correctly. Currently, the ones in the guide are misaligned and the ones on the downloads page are overlapping images in undesirable ways. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I made those final changes. The pages are live here: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Also, I normalized the text of the buttons so that each is the same across the two pages on which it is used. The only discrepancy is between the EV3's buttons on the two pages, where one is prefixed by "Download for" to make the intent of the buttons clear. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
One thing I want to do now that we are starting to have stretch images is make the download buttons say "Download ev3dev-jessie for LEGO MINDSTORMS EV3". |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I just had a look at the previews and "ev3dev-jessie" is in the file name, so that might be good enough. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
By themselves, the buttons look great, but in the context of the page, I think the gray makes them too busy and makes them look a bit out of place. It is especially bad on the getting started page with the alerts above and below. What if we just keep everything orange? Then the buttons won't be so tall either. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Wasn't the original issue that the white badge was too distracting on the plain orange buttons? For comparison, here's a CSS-ed screenshot of roughly what the normal button would look like with the new layout: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I could do without the badge regardless of the background color. Stylizing the file size with italics or something else would be just fine with me. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.