Task #40852

Story #40628: As any user, I need a Content Module based on semantic technologies (IKS)

Button-Bar should be positioned using jQuery UI position

Added by Sebastian Kurfuerst over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
0.00 h

Description

Currently the handles to add/delete/move content are displayed manually inside ui/elements/contentelement-handles.js around line 61.

However, this seems to break depending on the site's CSS quite often (for example on the flow3.org website).

Thus, we want to replace the manual positioning with using jQuery UI position helper: http://jqueryui.com/demos/position/#events

Bonus: If possible make the handles appear only when the content element is active, and not on hover. This should make the UI more visually "stable".

#1

Updated by Sebastian Kurfuerst over 8 years ago

Currently, they are just manually positioned (see ui/elements/contentelement-handles.js around line 61). However, that is prone to errors and often breaks the positioning.

We'd like to use jQuery UI Position widget for that, as that's hopefully more robust.

#2

Updated by Sebastian Kurfuerst over 8 years ago

  • Status changed from New to Accepted
#3

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14582

#4

Updated by Mattias Nilsson over 8 years ago

  • Status changed from Accepted to Under Review
#5

Updated by Sebastian Kurfuerst over 8 years ago

  • Status changed from Under Review to Accepted
#6

Updated by Sebastian Kurfuerst over 8 years ago

  • Assignee changed from Mattias Nilsson to Sebastian Kurfuerst
#7

Updated by Sebastian Kurfuerst over 8 years ago

I'll take over this task, and check it in various sites

#8

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14582

#9

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14582

#10

Updated by Gerrit Code Review over 8 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14582

#11

Updated by Gerrit Code Review over 8 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14582

#12

Updated by Sebastian Kurfuerst over 8 years ago

  • Status changed from Accepted to Under Review
#13

Updated by Gerrit Code Review over 8 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14582

#14

Updated by Mattias Nilsson over 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Applied in changeset commit:fb3ec69f5ce37d8459e9278658625ea34bbf01b8.

#15

Updated by Markus Goldbeck over 8 years ago

  • Status changed from Resolved to Closed
  • Estimated time set to 0.00 h
  • translation missing: en.field_remaining_hours set to 0.0

Also available in: Atom PDF