-
Notifications
You must be signed in to change notification settings - Fork 93
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
Fix shifting div
from KTextbox
#279
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
possibly a simpler fix?
Co-authored-by: Devon Rueckner <[email protected]>
@indirectlylit That was definitely the simpler fix. Seems to fix the issues that were found in learningequality/kolibri#8675 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool, thank you.
before merging, would you mind adding an entry to the changelog?
Darn, I forgot about this one before I tagged 1.2.0! I've pushed updates and will make a 1.2.1 now. |
Description
Changes the
v-if
to thev-show
in the KeenUI textbox so that the element is already in the DOM and we can toggle itsdisplay
.This is useful because it will keep that
div
belowKTextbox
for error validation, and prevent the UI from "shifting" whether or not there is an error validation for the field.Issue addressed
div
shifts slightly when focus is onKTextbox
#238 in KDSKTextbox
in form vertical shift kolibri#8746Before/after screenshots
See learningequality/kolibri#8746
Steps to test
After linking KDS to Kolibri, see the following for testing: learningequality/kolibri#8746
Does this introduce any tech-debt items?
KTextbox
in form vertical shift kolibri#8746) are the only places where the shifting needs to be fixed.Testing checklist
Reviewer guidance
git commit
)Post-merge updates and tracking
Comments