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
feat(ProfileShowcase): Implement custom showcase position when searcher text is filtering items #14486
feat(ProfileShowcase): Implement custom showcase position when searcher text is filtering items #14486
Conversation
Jenkins BuildsClick to see older builds (12)
|
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.
LGTM
ui/app/AppLayouts/Profile/panels/ProfileShowcaseAccountsPanel.qml
Outdated
Show resolved
Hide resolved
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 really like the approach, it organizes things nicely. Only some small questions here.
And as a future ideas:
- bulk set(...) method for WritableModel - it would prevent from many dataChange signals and other reevaluations later in the flow
- custom proxy just for DnD, simpler than MovableModel, converting dataChanged from source into rowsMoved signals
ui/app/AppLayouts/Profile/helpers/VisibilityAndPositionDirtyStateModel.qml
Outdated
Show resolved
Hide resolved
8e3625a
to
be9d6e9
Compare
✔️ status-desktop/prs/linux/x86_64/tests-nim/PR-14486#2 🔹 ~6 min 40 sec 🔹 be9d6e9 🔹 📦 tests/nim package |
@micieslak Good point! I've updated this PR and created two additional tasks: Feel free to update them. |
be9d6e9
to
800fa13
Compare
…er text is filtering items
800fa13
to
8185f1d
Compare
What does the PR do
closes #13682
closes #14040
Changes:
VisibilityAndPositionDirtyStateModel.qml
Affected areas
Profile showcase
Screenshot of functionality (including design for comparison)
Screen.Recording.2024-04-19.at.12.47.53.mov