Skip to content
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

WIP: Some investigations with Qt 6.5.3 [DO NOT MERGE] #2895

Closed
wants to merge 1 commit into from

Conversation

PeterPetrik
Copy link
Contributor

@PeterPetrik PeterPetrik commented Nov 1, 2023

Some investigations with Qt 6.5.3 on iOS

#2893

  • Repeater misbehaves, when you filter in Q sort model, the change of rowCount() is not applied to repeater, so it will have some zombie elements. Interestigly beginModelReset/endModelReset fixes it, or also invalidate() calls
    (fixed in this PR atm)

#2892

  • this is ScollView -> ColumnLayout, scrolling is possible but always pan back to top - not sure why
  • it works if I remove Repeater and add there only simple elements
  • it looks Repeater may have 0 implicitHeight
    (NOT fixed in this PR atm)

#2887

  • it looks like ScrollView -> Column is not possible to scroll on iOS, replacing with ColumnLayout enables scrolling
    (somewhat fixed in this PR atm)
  • we have also this is the project errors dialog and maybe somewhere else too

@PeterPetrik PeterPetrik changed the title Some investigations with Qt 6.5.4 WIP: Some investigations with Qt 6.5.4 [DO NOT MERGE] Nov 1, 2023
@PeterPetrik PeterPetrik changed the title WIP: Some investigations with Qt 6.5.4 [DO NOT MERGE] WIP: Some investigations with Qt 6.5.3 [DO NOT MERGE] Nov 1, 2023
@PeterPetrik PeterPetrik mentioned this pull request Nov 2, 2023
@PeterPetrik
Copy link
Contributor Author

Reverting to qt652, closing this for now

@PeterPetrik PeterPetrik closed this Nov 2, 2023
@PeterPetrik PeterPetrik deleted the fix_workspace_list branch March 20, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants