Skip to content

[FIX] Adds css styling to the cmsPage.html#13626

Closed
fliptrail wants to merge 5 commits intoRocketChat:developfrom
fliptrail:cms-page-edit
Closed

[FIX] Adds css styling to the cmsPage.html#13626
fliptrail wants to merge 5 commits intoRocketChat:developfrom
fliptrail:cms-page-edit

Conversation

@fliptrail
Copy link
Contributor

@fliptrail fliptrail commented Mar 6, 2019

Makes the Terms of Condition page and the Privacy Policy page more attractive to the user.

Closes #13577

Initial State:
imported

Final State:

Updated the CSS styling like this so that headings and text are clearly visible. Also added margins and padding to the whole container.
screenshot from 2019-03-07 20-58-06

Makes it look more user friendly
@codekhal
Copy link

codekhal commented Mar 6, 2019

It Looks decent but you could add more space between the last 2 lines and improve a bit of styling so issue hopefully would be closed. Though, it's just my opinion.
Cheers :)

@fliptrail
Copy link
Contributor Author

Problem with that is that the text coming from server is not marked up properly. There's a brute force method. Will try to use it.

@fliptrail
Copy link
Contributor Author

Added more commits and improved the final status

@fliptrail
Copy link
Contributor Author

Please review this PR

@fliptrail
Copy link
Contributor Author

@tassoevan @wreiske Please review this.

@tassoevan tassoevan requested review from ggazzo and tassoevan March 11, 2019 16:13
@tassoevan tassoevan added type: bug area: ui Touches the code on client side labels Mar 11, 2019
@ggazzo ggazzo added this to the 1.0.0 milestone Mar 12, 2019
@ggazzo
Copy link
Member

ggazzo commented Mar 12, 2019

@fliptrail sorry but you deleted your branch I cant push any fix, and I dont think its good enough... sorry...

@ggazzo ggazzo closed this Mar 12, 2019
@fliptrail
Copy link
Contributor Author

I will try again better next time. Actually this was one of my first PR's on this repo and I was actually trying to understand the repo structure.

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

Labels

area: ui Touches the code on client side type: bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default styles for Terms of Service and Privacy Policy are hideous

4 participants

Comments