U4-7411 - Upgrade plugin script for slider property editor

Created by Bjarne Fyrstenborg 16 Nov 2015, 13:53:41 Updated by Sebastiaan Janssen 05 Jan 2016, 10:09:07

Relates to: U4-4932

The Slider property editor is using the bootstrap slider plugin (v2.0.0): http://www.eyecon.ro/bootstrap-slider/

https://github.com/umbraco/Umbraco-CMS/tree/d50e49ad37fd5ca7bad2fd6e8fc994f3408ae70c/src/Umbraco.Web.UI.Client/src/views/propertyeditors/slider https://github.com/umbraco/Umbraco-CMS/blob/d50e49ad37fd5ca7bad2fd6e8fc994f3408ae70c/src/Umbraco.Web.UI.Client/lib/slider/js/bootstrap-slider.js

However since then there is a much newer version, which seems to work very well on touch devices and it has new properties, functions and events: https://github.com/seiyria/bootstrap-slider (v.5.2.6) The current version of the plugin in Umbraco 7 has some issues on touch devices and the tooltip does not always display the right value.

demo of the newer version: http://seiyria.com/bootstrap-slider/

Some cool new features added:

  • Ticks
  • Choose to always display tooltip
  • Coloring the low and high track segments (Example 12) - color of "high track", "low track" and "selection" (we can add config properties to specify each of these colors).

Maybe we can also add the option for start and end labels like Example 2 here: http://seiyria.com/bootstrap-slider/ and the option to specify the width - at least on mobile and tablet we should probably use full width (in horizontal mode), to make it easier to set the right value.

Comments

Bjarne Fyrstenborg 19 Nov 2015, 00:23:46

Using the "precision" property in v5 of the script, it should be possible to fix http://issues.umbraco.org/issue/U4-4932

https://github.com/bjarnef/Umbraco-CMS/commit/111e41c0440becbdd7e8509bca847b16d6304764#diff-632eb04d49bae992fd633720f6a192d7R95


Bjarne Fyrstenborg 23 Nov 2015, 01:40:44

PR: https://github.com/umbraco/Umbraco-CMS/pull/924


Claus Jensen 15 Dec 2015, 14:45:15

Tested and ensured it also works with existing sliders configured in backoffice. Added a little localization to some required fields and merged in PR.

Thanks for your help @Bjarne.Fyrstenborg!


Priority: Normal

Type: Feature (request)

State: Fixed

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted: Pull request

Affected versions: 7.3.0, 7.3.1

Due in version: 7.4.0, 7.3.5

Sprint: Sprint 5

Story Points:

Cycle: