janitor icon indicating copy to clipboard operation
janitor copied to clipboard

Tweaks to be done after converting all the pages to new design

Open ar5had opened this issue 8 years ago • 8 comments

Issues:

  • [ ] Active a color
  • [ ] Link gets hidden below header on blog and api page
  • [ ] Contrainers and settings page mobile layout
  • [ ] Tabs in mobile view when width is not sufficient
  • [ ] Favicon is not crisp
  • [ ] Typography tweaks
  • [ ] Projects page - timestatus and button edgecases
  • [ ] Projects page - new container button placement
  • [ ] Projects page - add border(#dedede solid 1px) to card
  • [ ] active menu item i.e., when the current page is in header menu (mobile view)
  • [ ] space between header and first menu item in mobile/tab view
  • [x] Blog page - clicking on link should expand that blog
  • [ ] Blog page css uses css variables
  • [x] ES6 to ES5
  • [ ] Move container class from main
  • [x] Buttons shadow get cropped on pages like blog-new
  • [ ] Sticky header on iphone&mac - chrome and safari
  • [ ] remove dotted outline of anchors visible on clicking in FX
  • [ ] alert color, font-size(sm-screens) and background color
  • [ ] decrease the size of pen icon on containers page
  • [x] Check all pages in tab and mobile view. Some page's layout breaks on sm screen e.g., containers, settings page
  • [ ] Change Delete ${containerName} to Delete Container on new-containers page.

ar5had avatar Feb 03 '18 09:02 ar5had

@arshdkhn1 Why do you want to " Move container class from main" ?

nt1m avatar Feb 06 '18 11:02 nt1m

Also, why do you want a border around project cards ? I think the box-shadow is enough.

nt1m avatar Feb 06 '18 11:02 nt1m

@nt1m

@arshdkhn1 Why do you want to " Move container class from main" ?

image

checkout button && also for landing page I was thinking to use full(100vw) width for some section but since parent wrapper is .container I can't do that.

Also, why do you want a border around project cards ? I think the box-shadow is enough. On chrome and mobile the box-shadow is a bit less prominent.

image

After adding a border(#dedede99 solid 1px) -

image

I think edges get a bit sharp and they dont blend with background.

Edit: On mobile,

chrome image

On chrome, I feel top part of box-shadow gets a bit merged with background. These results may vary on your screen, I am not sure about this.

ar5had avatar Feb 06 '18 17:02 ar5had

@arshdkhn1 It is definitively a bug, however, I don't think moving the .container class away from main is the way to go however.

As for the border, the shadow is actually stronger on macOS, not sure why Linux renders it like this. Anyway, I really don't like the border. It looks unclean to me. I prefer it without a border. I would rather tweak the box-shadow to be slightly stronger than add a border.

nt1m avatar Feb 06 '18 17:02 nt1m

@nt1m well the box-shadow issue can be managed like u did on project/container page? how do i get to have 100vw width for sections? I know adding .container everytime is redundant but I would love to hear other solutions?

ar5had avatar Feb 06 '18 17:02 ar5had

@arshdkhn1 I checked all the pages with Firefox's Responsive Design Tool, I think it looks great (so I marked the mobile device review in your checklist):

Landing page: main


Projects page: projects


Containers page (Info tab): containers


Containers page (Advanced tab): containersadvanced


Settings page (Account tab): settings


Settings page (Configurations tab): settingsconfigurations

Best regards,

Coder206

Coder206 avatar Jul 31 '18 00:07 Coder206

@Coder206 thanks!

ar5had avatar Jul 31 '18 02:07 ar5had

What are the planned "typography tweaks", anyway?

notriddle avatar Aug 23 '18 21:08 notriddle