-
Notifications
You must be signed in to change notification settings - Fork 315
Revamp examples page #2583
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
base: master
Are you sure you want to change the base?
Revamp examples page #2583
Conversation
14ba070 to
a1273b1
Compare
c795a4f to
5e1fac5
Compare
|
Missing tooltips, dangit. I'll add that in right away. |
|
Hi, thanks for this proposal @HoloTheDrunk , it is way better and more usable ! After discussion with @clameger , we have a few modification propositions:
|
d15c178 to
902a886
Compare
902a886 to
29a429e
Compare
|
Hi @HoloTheDrunk, Sorry for the late feedback but I finally took some time to do some more tests. A few propositions:
A UI designer from my team will also work on a new UI, including a gallery as mentioned in my previous comment but she will only be able to work on this in February. Therefore I propose that we merge this PR when the small changes mentioned in the list above are done and we work on the UI in another PR. |
Requires #2590
Description
Redesign of the examples page:
<iframe>instead of a direct link to the example blob on GitHub.Motivation and Context
The examples page is one of the first things people will see when researching which library to use. By providing immediately editable examples, we provide a way for prospective users to toy around and see if iTowns matches their needs for a very low time investment on their end.
Things to be done after merge
Screenshots
1080p full width



Lower resolution / reduced width
Mobile ratio (editor shows up in landscape mode)