-
Notifications
You must be signed in to change notification settings - Fork 48
Improve Design and Layout #352
Comments
Redesign of Options page. Reasons to change:
Some more suggestions:
|
Hi @prastut! I think this is an improvement, so long as you implemented it with Bootstrap's responsive columns. Want to open a PR? |
@smcgregor I would love to but I am facing a little difficulty in understanding the workflow. I will elaborate the steps, correct me where I am wrong.
I think I have some misunderstanding since the dev pages say to clone the platform related repo (in my case chrome) recursively. |
You should clone one of the projects that incorporate privly-applications as a submodule. When you use the |
@smcgregor Makes sense! Thanks :D I cleaned my branch but had a doubt. If I build the system, there are a lot changes shown amidst the ones I actually did. Is this normal? |
@prastut That is normal. Do not commit any built html files, just the templates. |
Please review PR #357. Should I rework the rest of the pages mentioned in the issue or leave it for the others who would be coming in, and move to either another Level1 issue or Level2 issue? Let me know. |
You can make more templating changes if you like, but it is generally good to move on to the higher levels if you understand what is happening with the lower ones. |
(This is intended to be an introductory issue, teaching new people how to use the build system)
Some options for making improvements are:
I am intentionally being vague in what can be improved since you are new to the project and will have better ideas of what needs improvement than someone who has already contributed. If you want to fix something specific, then I suggest you move the information block so it snaps to the footer of the page instead of floating up to meet the content. You can see this on the "New Message" page, when you make the window tall.
You may need to read about Twitter Bootstrap to make changes. Bootstrap is a "responsive" library, meaning it helps change the layout of the page in response to many different screen resolutions.
The text was updated successfully, but these errors were encountered: