Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

WIP: feat(component): update styling of property pane #373

Closed
wants to merge 35 commits into from

Conversation

markusoelhafen
Copy link
Member

@markusoelhafen markusoelhafen commented Apr 13, 2018

This PR replaces #371 .
I started to update the UI components for alva, to enhance usabilty

Tasks

  • Styling of String Property
  • Styling of Enum Property
  • Styling of Boolean Property
  • Styling of Asset Property
  • Styling of Unknown Property

border-radius: 3px;

background-color: ${colors.white.toString()};
background-image: url("data:image/svg+xml;utf8,<svg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 24 24' fill='${colors.grey60.toString()}'><path d='M8,4 l8,8 -8,8 z' /></svg>");
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added the icon as a background image (still wrong rotation) for now, as I haven't found a solution on how to import the icon component for this issue. Is there any way to resolve this @Palmaswell @lkuechler ?

@tilmx tilmx changed the title WIP: Feature / UI improvements WIP: feat(component): update styling of property pane Apr 23, 2018
@tilmx
Copy link
Member

tilmx commented Apr 23, 2018

I started a pull request for the chrome/element pane updates in #393.

@tilmx tilmx force-pushed the feat/ui-improvements branch from 2058336 to 047783b Compare April 23, 2018 09:32
@marionebl
Copy link
Contributor

Closing due to #393 superseeding this.

@marionebl marionebl closed this Apr 25, 2018
@tilmx
Copy link
Member

tilmx commented Apr 27, 2018

#393 is about the Layout and Element Pane, this one is about the Property Pane, so it should be still valid.

@tilmx tilmx reopened this Apr 27, 2018
@tilmx
Copy link
Member

tilmx commented May 7, 2018

Superseeded by #436

@tilmx tilmx closed this May 7, 2018
@tilmx tilmx deleted the feat/ui-improvements branch May 24, 2018 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants