-
-
Notifications
You must be signed in to change notification settings - Fork 243
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
Items list: Fix searchbar broken after Items list update & Fix search query not stored #2184
Conversation
Fixes openhab#1334. Fixes openhab#1702. After editing or removing an Item, the searchbar of the Items list was not working anymore. Console showed this error: `TypeError: undefined is not an object (evaluating 'n.virtualList.params')` It seems that this was caused because the replaceAllItems operation was applied on the virtual list, but the was searchbar not re-rendered afterward. Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
…enter Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Job #1289: Bundle Size — 15.82MiB (+0.34%).Important Bundle introduced 1 and removed 1 duplicate package – View changed duplicate packages Warning Bundle introduced 13 new packages: @jsep-plugin/regex, @jsep-plugin/arrow, @jsep-plugin/object and 10 more – View changed packages Bundle metrics
Bundle size by type
View job #1289 report View florian-h05:items-list-searchbar branch activity |
/cc @J-N-K |
I can confirm this fixes the issue. |
Great! @ghys Can you please review this PR? |
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.
Code looks good to me, Florian
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.
I'm not sure about the storage in the Vuex store, see below - otherwise LGTM in principle.
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@ghys I addressed your review regarding the Vuex store, so this should be ready for merging! |
Fixes #1334.
Fixes #1702.
After editing or removing an Item, the searchbar of the Items list was sometimes not working anymore. Console then showed this error:
It seems that this was caused because the replaceAllItems operation was applied on the virtual list without re-rendering the searchbar afterward.
The search query was forgotten when the Items list reloaded or the Items list page was re-entered from an Item detail page.
This is also fixed by using
$f7.data
to store the query.