architect icon indicating copy to clipboard operation
architect copied to clipboard

remove text-indent for headings in mobile

Open paradite opened this issue 4 years ago • 1 comments

Problem

text-indent was introduced for compensating the blue ticks before the headings https://github.com/pages-themes/architect/blob/27f3b20dfb55063555c3f029dcffe75f3d067fb2/_sass/jekyll-theme-architect.scss#L197

They were not removed in mobile, whereas the blue ticks were removed https://github.com/pages-themes/architect/blob/27f3b20dfb55063555c3f029dcffe75f3d067fb2/_sass/jekyll-theme-architect.scss#L377-L379

This results in extra indentation in mobile.

Solution

This css change removes unnecessary text-indent in mobile.

Before / After

Using my own page because I am a bit lazy for reproducing this...

before after
Screenshot_20211219-144322_Chrome - Copy (2) Screenshot_20211219-144355_Chrome (2)

live website effect reference: https://retire35.com/

paradite avatar Dec 19 '21 07:12 paradite

Welcome! Congrats on your first pull request to the Architect theme. If you haven't already, please be sure to check out the contributing guidelines.

welcome[bot] avatar Dec 19 '21 07:12 welcome[bot]