Page MenuHomePhabricator

ve.ui.Toolbar should have a configurable offset for fixed headers
Closed, ResolvedPublic1 Estimated Story Points

Description

When you scroll in Vector with VE enabled, the toolbar will be fixed to the top of the window if the toolbar reach it. This works fine, when you have a skin without a fixed header, but some skins have a fixed header, so the VE toolbar is (maybe partially) hidden by a fixed element. This could be solved by implementing a configurable offset (e.g. using a JS variable, which isn't set in VE, but can be set by any extension/skin), which can be set by skins, that need it. This would be one step into the direction to support more skins.

Event Timeline

Florian claimed this task.
Florian raised the priority of this task from to Needs Triage.
Florian updated the task description. (Show Details)
Florian added a project: VisualEditor.
Florian added a subscriber: Florian.

Change 203010 had a related patch set uploaded (by Florianschmidtwelzow):
Allow to fix the toolbar not to the top of the window

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

Change 203011 had a related patch set uploaded (by Florianschmidtwelzow):
Allow skin developers to set an offset for toolbar floating

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

Change 203010 merged by jenkins-bot:
ve.ui.Toolbar: Allow setting positioning offset from the top of the window

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

Change 203011 merged by jenkins-bot:
Allow skin developers to set an offset for toolbar floating

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