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.

Sign up for GitHub

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

Jump to bottom

Create a Sortindex #123

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Create a Sortindex #123

wants to merge 9 commits into from

Conversation

ArneGudermann
Copy link
Contributor

Replace a Numericbone if the Bonename is "sortindex". Make an Listview sortable.

ArneGudermann and others added 7 commits March 16, 2022 14:27
@ArneGudermann
Add Sortindex Check
840d201
@ArneGudermann
Add ModuleName
82707c1
@ArneGudermann
Chnage Sort
cbf0753
@ArneGudermann
Reformat Code
d1e67c2
@phorward
Update datatable.py
feeb978 
Cleared syntax and code-style a bit
@phorward
Update datatable.py
33f0ebb 
Changed order of sortindex evaluation improving runtime performance
@phorward
Update datatable.py
36add91 
GitHub Online Editor made bullshit
@phorward
Copy link
Member

Hello @ArneGudermann please keep this PR open, as it was now used in a project where necessary.
I was not happy with the implementation, as it should become a bone widget as it was the case in the plugin from VI2.
Nevermind, just keep it open and we should integrate the feature into VI4 as well.

@phorward phorward reopened this Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akelch akelch Awaiting requested review from akelch

@phorward phorward Awaiting requested review from phorward

@skoegl skoegl Awaiting requested review from skoegl

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ArneGudermann @phorward