search

housseindjirdeh / angular2-hn

:boom: Progressive Hacker News client built with Angular
https://angular2-hn.firebaseapp.com
MIT License
1.3k stars 233 forks source link

Enhancing the design of the settings popup #78

Closed portexe closed 4 years ago

portexe commented 5 years ago

There are a couple of changes made here.

  1. I fixed the layout of the HTML to be much more clean.

  2. I added a class called control-section and encapsulated all of the control sections inside of a div with this class.

  3. I made the text inputs display: block. I also set a width so that they would be uniform. Added a border radius and a little padding for aesthetic.

  4. I gave the control-sections some padding and margin, along with a bottom border so that they would be easier to see the separation.

housseindjirdeh commented 4 years ago

I'm sorry for being so late 😅

I kind of stopped maintaining this but trying to clean up a bunch of old issues and PRs. Thank you so much for taking the time to do this.