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

feat: update styling of preview resizing #410

Merged
merged 5 commits into from
May 3, 2018
Merged

Conversation

markusoelhafen
Copy link
Member

clean up the preview pane ui and make resize elements visible on hover.
let me know what you think @tilmx

@tilmx
Copy link
Member

tilmx commented May 3, 2018

I really like the styling and the UX itself. The only thing I am a bit unsure is about if it is so good to have those spaces not clickable for the preview content itself. So e.g. if on the website there is something to be clicked on the left or right side, then you can’t click that any more because of that handles. But let’s talk later about that @markusoelhafen!

@tilmx tilmx changed the title Feat/resize pattern feat: update styling of preview resizing May 3, 2018
Copy link
Member

@tilmx tilmx left a comment

Choose a reason for hiding this comment

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

Let’s do that!

@tilmx tilmx merged commit 8b8a19c into master May 3, 2018
@tilmx tilmx deleted the feat/resize-pattern branch May 3, 2018 20:51
Palmaswell pushed a commit that referenced this pull request May 7, 2018
* fix(component): update resize pattern ux
* fix(app): remove unneeded comma
* fix(component): adjust resize hover styling
* fix(component): add transition on hover
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.

2 participants