Skip to content

Conversation

amolenaar
Copy link
Member

@amolenaar amolenaar commented Jul 7, 2023

PR Type

What kind of change does this PR introduce?

  • Bug fix
  • Feature
  • Chore (refactoring, formatting, local variables, other cleanup)
  • Documentation content changes

What is the current behavior?

The editable label causes crashes when used in a ListView, or so it seems.

Issue Number: #2433

What is the new behavior?

Revert to Gtk.TreeView. We need to find out why it's crashing and how to fix it.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

This reverts commit c291034, reversing
changes made to d7ee475.
@amolenaar amolenaar requested a review from danyeaw July 7, 2023 21:17
@github-actions github-actions bot added the python Pull requests that update Python code label Jul 7, 2023
@danyeaw danyeaw added bug An issue in the application and removed python Pull requests that update Python code labels Jul 8, 2023
@danyeaw danyeaw merged commit f4e882e into main Jul 8, 2023
@danyeaw danyeaw deleted the revert-parameter-listview branch July 8, 2023 01:19
@amolenaar
Copy link
Member Author

Should we do a new release due to this error?

amolenaar added a commit that referenced this pull request Jul 8, 2023
There should be a way to make this work safely.
@danyeaw
Copy link
Member

danyeaw commented Jul 8, 2023

Ya, I think so as well

amolenaar added a commit that referenced this pull request Sep 16, 2023
There should be a way to make this work safely.
amolenaar added a commit that referenced this pull request Dec 15, 2023
There should be a way to make this work safely.
amolenaar added a commit that referenced this pull request Dec 15, 2023
There should be a way to make this work safely.
amolenaar added a commit that referenced this pull request Dec 17, 2023
There should be a way to make this work safely.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue in the application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants