Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Make AppsDrawer resizable by dragging its bottom border #4493

Closed
wants to merge 4 commits into from

Conversation

pv
Copy link
Contributor

@pv pv commented Apr 25, 2020

Make widgets resizable by dragging their bottom border with mouse. Implements element-hq/element-web#6785 (see especially element-hq/element-web#6785 (comment))

Video

Fixes element-hq/element-web#6785

@turt2live turt2live requested review from turt2live and a team April 27, 2020 17:31
@turt2live turt2live added the Z-Community-PR Issue is solved by a community member's PR label Apr 27, 2020
@nadonomy
Copy link
Contributor

@pv is it possible to make the target to drag more generous? From the video it looks like your cursor struggles to drag exactly, toggling between cursors rapidly.

pv added 3 commits April 30, 2020 20:14
Using flexboxes removes the need for pixel constants, and the app
content now resizes to fill the tiles.

Signed-off-by: Pauli Virtanen <pav@iki.fi>
The event coordinates are document coordinates, so the offset they are
compared to should also be the document one.

Signed-off-by: Pauli Virtanen <pav@iki.fi>
Signed-off-by: Pauli Virtanen <pav@iki.fi>
@pv pv force-pushed the resizable-appsdrawer branch from 0acb5c9 to 9138e9e Compare April 30, 2020 17:35
@pv pv force-pushed the resizable-appsdrawer branch from 9138e9e to 7358260 Compare April 30, 2020 17:35
@pv
Copy link
Contributor Author

pv commented Apr 30, 2020

@nadonomy: thanks, the handle indeed was half hidden behind the apptile. Fixed it, now it should be the same size as the other handles: https://raw.githubusercontent.com/pv/matrix-react-sdk/resizable-appsdrawer-media/appsdrawer-2.webm

@turt2live turt2live removed their request for review May 19, 2020 21:29
@nurjinjafar
Copy link
Contributor

@t3chguy
Copy link
Member

t3chguy commented Jul 27, 2020

@nurjinjafar sounds like your browser doesn't support WebM. It plays fine in Firefox

@t3chguy
Copy link
Member

t3chguy commented Aug 24, 2020

Has been superceded by #5138, making use of the above commits as a foundation.

@t3chguy t3chguy closed this Aug 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resize widget area height in room via Riot interface
5 participants