fix(VSlider): set value to min if it's less than min (#6604)

[hrayr-artunyan]

Description

Forces the slider value to be within the boundaries set by min and max props.

Motivation and Context

fixes #6604

How Has This Been Tested?

Tested visually with the provided playground code.

Markup:

<template>
  <v-app>
    <v-layout justify-center wrap>
      <v-flex xs7>
        <v-slider
          v-model="val"
          min="55"
          max="80" />
      </v-flex>
    </v-layout>
  </v-app>
</template>

<script>
  export default {
    data: () => ({
      val: 50
    }),
  }
</script>

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

Checklist:

• The PR title is no longer than 64 characters.
• The PR is submitted to the correct branch (master for bug fixes and documentation updates, dev for new features and breaking changes).
• My code follows the code style of this project.
• I've added relevant changes to the documentation (applies to new features and breaking changes in core library)

[nekosaur]

Thank you for your contribution. Could you possibly create a test for this as well?

[dsseng]

Looks good. Please add unit test for this.

[codecov]

Codecov Report

Merging #6605 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #6605   +/-   ##
=======================================
  Coverage   85.69%   85.69%           
=======================================
  Files         298      298           
  Lines        7173     7173           
  Branches     1784     1784           
=======================================
  Hits         6147     6147           
  Misses        935      935           
  Partials       91       91

Impacted Files	Coverage Δ
packages/vuetify/src/components/VSlider/VSlider.js	99.33% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 592f9f0...6af735f. Read the comment docs.

[codecov]

Codecov Report

Merging #6605 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #6605   +/-   ##
=======================================
  Coverage   85.68%   85.68%           
=======================================
  Files         298      298           
  Lines        7175     7175           
  Branches     1785     1785           
=======================================
  Hits         6148     6148           
  Misses        936      936           
  Partials       91       91

Impacted Files	Coverage Δ
packages/vuetify/src/components/VSlider/VSlider.js	99.33% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67b73b9...13ecbbd. Read the comment docs.

[dsseng]

@hrayr-artunyan please add unit tests!

[hrayr-artunyan]

@sh7dm I will do so shortly. Thank you.

needs unit tests

[hrayr-artunyan]

@nekosaur and @sh7dm I finally got around to adding a unit test.

Please test, thanks!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. Please direct any non-bug questions to our Discord