Skip to content

Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix +Disabled Switch CSS fix + Sidebar ClickNotClose Issue fixed #513

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Mar 27, 2019

Conversation

jd-solanki
Copy link
Contributor

@jd-solanki jd-solanki commented Mar 23, 2019

I have added functionality to update page number from outside of pagination component.
Here's what it looks like.

chrome-capture (1)

I have tested it there's no bug even if there's negative value or value is greater than total pages.

This solves my issue
i also fixed checkbox issue & This one
New Fix: Fixed disabled switch CSS issue
Sidebar component's click-not-close prop was broken. Fixed.

@jd-solanki jd-solanki changed the title Added functionality of programmatic Inc/Dec Page Number + Update Docs Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix Mar 26, 2019
@jd-solanki jd-solanki changed the title Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix +Disabled Switch CSS fix Mar 26, 2019
@jd-solanki jd-solanki changed the title Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix +Disabled Switch CSS fix Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix +Disabled Switch CSS fix + Sidebar ClickNotClose Issue fixed Mar 26, 2019
@luisDanielRoviraContreras luisDanielRoviraContreras merged commit 8418e44 into lusaxweb:master Mar 27, 2019
@@ -92,7 +92,7 @@ export default {
this.setValueString()
}
else {
this.$emit('input', evt.target.value)
this.$emit('input',!this.value)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does NOT work with what you updated in this line

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested that in doc. In my clone it worked so. Anyway, I will try once again. Because my project is running in production can you please find solution to this first. Because this is MAJOR issue.
Waiting for fix.

Thanks

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is already solved apparently there is a conflict when adding input and change and they run together

Tofandel pushed a commit to Tofandel/fixed-vuesax that referenced this pull request Apr 26, 2021
Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix +Disabled Switch CSS fix + Sidebar ClickNotClose Issue fixed
Tofandel pushed a commit to Tofandel/fixed-vuesax that referenced this pull request Apr 26, 2021
Added functionality of programmatic Inc/Dec Page Number + Update Docs + Checkbox fix +Disabled Switch CSS fix + Sidebar ClickNotClose Issue fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants