ekke / drawer_nav_x Goto Github PK

Hi, Ekkehard

Just a little suggestion-improvement for your QTBUG-54260 workaround.

For now you store properties
string name: "namePage"
for every single page and then set or restore SwipeView index looking by name "colorSchemaNavPage" that as you know contains SwipeView:
if(rootPane.currentItem.name == "colorSchemaNavPage")

Despite this is just a workaround, I think, you could improve it a little just by checking lastCurrentIndex property existence:
rootPane.currentItem.lastCurrentIndex != undefined

That's it. No need to store names properties for any page (at least to use workaround). No need to add new names of pages with SwipeView to checking condition if somebody decides to add some of them (and will have to rack his brains why SwipeView currentIndex lost only on his new pages).

p.s. Taking this opportunity I want to thank you for your blog and applications' code sharing!