-
Notifications
You must be signed in to change notification settings - Fork 258
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: Vue3 => Error info when creating NavLink is not shown to user #11956
base: master
Are you sure you want to change the base?
FIX: Vue3 => Error info when creating NavLink is not shown to user #11956
Conversation
30094bb
to
c14a0c4
Compare
@richard-cox added you as a reviewer not only because I couldn't find the exact root cause but because of the implications this might have on other areas and potencial checks to be done. Let me know if you want to get together to look into this |
This looks like another instance where |
@rak-phillip I am interested in knowing more about the root cause of this.... I never understood why adding a root element would fix this 😛 |
@aalves08 this all has to do with how Vue3 manages fallthrough attributes1 for root-level ancestor components - I've created a minimal repro to demonstrate the behavior:
This can be fixed in several ways:
Footnotes |
Summary
Fixes #11937
Occurred changes and/or fixed issues
Technical notes summary
Although the changes look bigger, what was needed to make it work was just adding a
div
as the outermost container to wrap theCruResource
component. No errors were being thrown, so I don't know the root cause for this.Also, it may be important to cover all of the
CruResource
usages to check if it's a general problem or not and act accordingly.Areas or cases that should be tested
Areas which could experience regressions
Screenshot/Video
Screen.Recording.2024-09-18.at.14.35.40.mov
Checklist