Closed RouxRC closed 8 years ago
I'm not sure, personally I prefer having exposé first to have a better clue what the amendment is about, but having text first makes comparing similar amendments easier...
well, IMHO it depends on your law knowledge : if I know the law very well, then I would prefer to see the text first. But if I know almost nothing about the law, I would prefer to read the expose as I would not understand the text.
so finally it really depends on the user. Maybe we should keep the exposé first as we want to target a global audience.
i guess that was the original intention, then maybe we could have some kind of way of always showing half expose half amendment, with a click needed to unfold the full expose when necessary
indeed, exposé can be quite long (and uninteresting :)) so maybe we can just show the first part (for example 250 characters) and let the user make a click to read the full exposé. As the text comes after, we can just show the full text
This would be far easier to make after #152. Can we postpone it?
+1 for leaving this feature after #152
After discussion with @teymour, we both agreed hiding part of the exposé makes sense, although, one important thing will be to let the text of the amendment always start at the same position so that you can keep your eye on the same level when switching from one to another. This means the Signataires part will also need to be handled together with the exposé
Re-fixed in fa8318c :)
I think we're nearly there, although I find the space allocated for the signataires way too big, only the height of 2 lines should be given as it is not important, can always be unfolded, and mostly leaves a big weird white space before we can read the actually interesting info
A few more things:
All fixed :)
strangely I still have the hand cursor on the unfolded zone on your dev instance
I do, too, maybe the travis job takes more time than usual to deploy. (The cursor: initial; directive for .expanded selector is not in app.css)
It's deployed correctly now ! nope, messed it up
mmm the new css still won't load on my machine but yes when i modify it it works, maybe we could try and force the css to be hard refreshed in some way? although i don't know any proper way to do that :/
NOW it's finally fixed
yeah!!!
@teymour was proposing it I feel like both ways have advantages Other views @fmassot @njoyard ?