Group-income-simple: Minor Design system fixes

Created on 19 Aug 2019  Â·  12Comments  Â·  Source: okTurtles/group-income-simple

Problem

I found some minor discrepancies between the design system on Figma and what was actually implemented. This list of changes was made after reviewing the app/design-system page in master (August 19, 2019).

Solution

  • [x] H1 and H2 should be responsive (figma);

  • [x] Fix small typo, on the subtitle typography atom. "This is an subtitle" should be "This is a subtitle";

  • [x] Buttons that have icons as a prefix or sufix should not have a margin on the left or a margin on the right side:
    Captura de ecrã 2019-08-19, às 16 52 54

  • [x] Fix small text movement in some inputs (dropdown, text + dropdown combo, text + button, text with icon prefix), when they change from their normal state to focus (see gif below - it takes a while to show the change):
    input-movement

  • [x] Remove hover CSS changes from the text input with icon prefix:
    Captura de ecrã 2019-08-19, às 19 56 11

  • [x] The subtitle on modals does not have the correct CSS - the subtitle component in the typography section is correct:
    Captura de ecrã 2019-08-19, às 20 11 24

  • [x] At the moment, it's possible to resize textareas horizontally. I understand that this is the default behaviour of textareas. The vertical resize is useful, but the horizontal one can be problematic. Is there a way to limit this movement? Bonus question, for 1000€ - how difficult is it to make a textarea increase its vertical size as the content needs it (aka, dynamically)? I found this codepen example

textarea2


  • [x] Update selectbox icon (chevron)
Frontend Bug

All 12 comments

I'll pick this stuff today, you should expect a PR later today ;)

I started to fix those already @sandrina-p

Oh okay, no problem! Next time @pieer let us know when you pick this, so we avoid doing the same thing twice :)

Let me know if you have any questions. By the way @pieer, I am using google chrome

That was mainly fix about my previous commit, so I assume I was going to fix it and you would focus on the sidebar, but yes, I will tell you next time

@pieer this is the issue being solved in PR #636 right?

Just reviewed this in master - most of these are solved with the exception of:

H1 and H2 should be responsive

Buttons that have icons as a prefix or suffix should NOT have a margin on the left or a margin on the right side


The first one (H1 and H2) are working as expected in desktop and tablet, but are missing the smaller breakpoint:

Captura de ecrã 2019-09-07, às 17 56 45


cc @pieer

Also noted that the dropdown input has the wrong icon. See below:
Captura de ecrã 2019-09-07, às 18 02 14
Captura de ecrã 2019-09-07, às 18 01 13

Added this one as a task as well

@mmbotelho which icon from fontawesome is that little triangle? (I'm realising I'm terrible looking for icons there)

Sorry for making you look there, it's not an icon - it's just a div shaped into a triangle (I believe this can be done with the :after pseudo-element or something similar). Here are the dimensions:

Captura de ecrã 2019-09-08, às 13 04 32

It is also placed slightly off-center (1 px closer to the bottom) or else it looks misaligned (because of the upper weight of the shape).

cc @sandrina-p

@pieer are you working on these?

I forgot the to change the icon ya

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mmbotelho picture mmbotelho  Â·  7Comments

taoeffect picture taoeffect  Â·  5Comments

taoeffect picture taoeffect  Â·  3Comments

sandrina-p picture sandrina-p  Â·  6Comments

taoeffect picture taoeffect  Â·  7Comments