Page MenuHomePhabricator

While using the resize handler on a graph, the lowest limit for resizing is 60x60px, but not when resizing from dialog
Closed, ResolvedPublic1 Story Points

Description

When I try to resize a graph using resize handler I seem to cant reduce it below 60x60 px but from the dialog it can go below that. Shouldnt they be same?

Details

Related Gerrit Patches:
mediawiki/extensions/Graph : masterVE: Use MediaSizeWidget with restricted dimensions

Event Timeline

Ryasmeen raised the priority of this task from to Needs Triage.
Ryasmeen updated the task description. (Show Details)
Ryasmeen added a project: VisualEditor.
Ryasmeen added subscribers: Ryasmeen, ferdbold.
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptNov 20 2015, 11:51 PM
Ryasmeen set Security to None.

Yeah me and @Esanders noticed that as well, that's because OO.ui.DimensionsWidget doesn't account for resizable nodes when it probably should.

Jdforrester-WMF renamed this task from While using the resize handler the lowest limit for resizing a graph is 60x60px but not when resizing from dialog to While using the resize handler on a graph, the lowest limit for resizing is 60x60px, but not when resizing from dialog.Nov 24 2015, 8:21 PM
Jdforrester-WMF assigned this task to ferdbold.
Jdforrester-WMF triaged this task as High priority.
Jdforrester-WMF edited a custom field.
Jdforrester-WMF moved this task from To Triage to Blocked on the VisualEditor board.

Change 257319 had a related patch set uploaded (by Esanders):
VE: Use MediaSizeWidget with restricted dimensions

https://gerrit.wikimedia.org/r/257319

Change 257319 merged by jenkins-bot:
VE: Use MediaSizeWidget with restricted dimensions

https://gerrit.wikimedia.org/r/257319