Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QUnit Adding Contribute ribbon link. #218

Closed

Conversation

kleinmaetschke
Copy link
Contributor

Attempting to fix #189.

Tossing my hat in the ring on a good way to display this. Corner ribbons were favored, so I went with that.
There were comments that the ribbon was a bit too large, so this is shrinking it a bit.
Moving to using an anchor for the ribbon instead of "jacky" javascripting.
@jzaefferer
Copy link
Member

This looks pretty good. There's still sizes at which the ribbon overlaps with small parts of the footer, but that might be harmless enough to ignore it.

Since the ribbon itself is pretty light on information about its purpose, how about adding a title attribute with something like "Report issues or contribute improvements on GitHub"?

@rxaviers
Copy link
Member

About the ribbon overlapping footer, I have a few suggestion for each width range:

  • [0px,425px] Overlaps.
    Possible solution: enough margin-bottom on "#legal", so it doesnt get covered.
    Captura_de_tela-5
  • [425px, 600px] No problem.
  • [800px, 1280px] Overlaps.
    Possible solution: enough margin-right on ".footer-site-links", so it doesnt get covered.
    Captura_de_tela-6

@rxaviers
Copy link
Member

If we still need to make it smaller, a slight-change-idea is to make it very minimal, but grow on hover.

@rxaviers
Copy link
Member

👍 on @jzaefferer's title idea. One more suggestion:
"Find any bugs or have ideas on how to make it even more amazing?"

PS: Once we define it, we could poke @ajpiano. He is a good person for those suggestions.

@scottgonzalez
Copy link
Member

Closing due to inactivity. I'd really like to see us move forward with a solution for #279, but this seems to have stalled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants