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

Make cards consistent again #1383

Merged
merged 55 commits into from Feb 14, 2018
Merged

Make cards consistent again #1383

merged 55 commits into from Feb 14, 2018

Conversation

taniki
Copy link
Contributor

@taniki taniki commented Jan 19, 2018

Based upon Improve Cards consistency but with features sorted out in various PR.

This one is only about a global overlook of look and feel of cards.

extracts

@taniki taniki self-assigned this Jan 19, 2018
@taniki taniki changed the title make cards consistent again [wip] make cards consistent again Jan 19, 2018
@noirbizarre
Copy link
Contributor

I think you need to rebase on the current master. The changelog seems to indicate that you are working a an old version of the master

Thomas Parisot and others added 24 commits January 19, 2018 16:48
Introduce the concept of concept colors. Each card type has its own distinct color.

It will be leveraged at the resource level in a further pull request
Also, signpost Community sections
It has too many side effects on the rest of the app otherwise.
@taniki
Copy link
Contributor Author

taniki commented Jan 22, 2018

@noirbizarre I have conflicts with your two last merges. I have integrated the second one but overrided part of the first one.

@abulte
Copy link
Contributor

abulte commented Feb 8, 2018

There are problems with some slides on the carousel with the default theme:

capture d ecran 2018-02-08 15 21 38

capture d ecran 2018-02-08 15 23 26

@abulte
Copy link
Contributor

abulte commented Feb 8, 2018

I wonder if we should have some different, more neutral colors for cards footer in the default theme. BTW, we agreed some time ago that the cards footer colors should be configurable, is this the case?

@noirbizarre
Copy link
Contributor

Former presentation is fine to be, but this is currently broken.

Incoherent on default theme too:

screenshot-demo data gouv fr-2018 02 08-15-33-56

@abulte
Copy link
Contributor

abulte commented Feb 8, 2018

There is a slight problem with tooltips on texts that overflow a lot (this is the tooltip of the left card):

capture d ecran 2018-02-08 17 26 14

http://demo.data.gouv.fr/fr/datasets/panorama-des-grands-projets-si-de-letat-1/

@abulte
Copy link
Contributor

abulte commented Feb 8, 2018

Slight problem between breakpoints on the admin (~1200px wide here):

capture d ecran 2018-02-08 17 38 57

https://demo.data.gouv.fr/fr/admin/editorial/

Proposed solution: only one item per row on both columns for all breakpoints.

@abulte
Copy link
Contributor

abulte commented Feb 8, 2018

Problem with dataset card on reuse page in the admin:

capture d ecran 2018-02-08 17 46 02

https://demo.data.gouv.fr/fr/admin/reuse/596f5fd5c751df02a36e4726/

@taniki
Copy link
Contributor Author

taniki commented Feb 8, 2018

The two previous bugs have been fixed. Can someone get a detailed look at the very last commit? I have change a stuff in the api and need an impact assesment

@taniki
Copy link
Contributor Author

taniki commented Feb 9, 2018

screen shot 2018-02-09 at 15 57 48

@abulte
Copy link
Contributor

abulte commented Feb 9, 2018

@taniki regarding your question, it might be better to add the fields you need to dataset_ref_fields instead of of getting all the fields. Which ones do you need?

@noirbizarre
Copy link
Contributor

Reuse cards from companies display None as author:
screenshot-demo data gouv fr-2018 02 12-17-21-21

@abulte
Copy link
Contributor

abulte commented Feb 13, 2018

Problems on user and org cards when creating a dataset:

capture d ecran 2018-02-13 16 03 07

capture d ecran 2018-02-13 16 03 17

@abulte
Copy link
Contributor

abulte commented Feb 13, 2018

Problem on topic admin view:

https://demo.data.gouv.fr/fr/admin/topic/53552f47c751df3ceca6f823

capture d ecran 2018-02-13 16 07 55

@abulte
Copy link
Contributor

abulte commented Feb 13, 2018

Misaligned close icon on topic dataset edition (clearly not critical):

capture d ecran 2018-02-13 16 10 02

@abulte
Copy link
Contributor

abulte commented Feb 14, 2018

Still a small glitch on user card (bottom right corner) when creating a dataset (can be fixed later):

capture d ecran 2018-02-14 09 26 29

Copy link
Contributor

@abulte abulte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@taniki
Copy link
Contributor Author

taniki commented Feb 14, 2018

@taniki taniki merged commit 25def3a into opendatateam:master Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants