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

tweak compact theme #4665

Merged
merged 1 commit into from
Aug 6, 2017
Merged

tweak compact theme #4665

merged 1 commit into from
Aug 6, 2017

Conversation

uhoreg
Copy link
Member

@uhoreg uhoreg commented Jul 24, 2017

Assigning to @ara4n so that he can decide if he likes the changes or not. ;) The reduced size of info event I think should be fine. I'm not so sure about the reduced spacing between some elements, because, at least for paragraphs, it comes into effect or not depending on how the user entered the paragraphs (see discussion starting at #4363 (comment)).

- add some comments
- decrease size of info events and MELS (element-hq#4363.2)
- reduce spacing between paragraphs, lists, etc (element-hq#4363.3 & 4)

Signed-off-by: Hubert Chathi <hubert@uhoreg.ca>
@ara4n
Copy link
Member

ara4n commented Aug 6, 2017

the compact theme is already treading its own aesthetic path, so I'm not sure my opinion counts for much :) hopefully the users will like the tweaks :D

@ara4n ara4n merged commit a98c734 into element-hq:develop Aug 6, 2017
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.

2 participants