Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[TreeView] Allow case-insensitive sorting of tree view keys #7188

Open
wants to merge 4 commits into
base: develop
Choose a base branch
Loading
from

Conversation

livecodepanos
Copy link
Contributor

@livecodepanos livecodepanos commented Oct 9, 2019

This patch adds a new boolean property sortCaseSensitive to the treeView widget, which defaults to true.

When true, the keys AA,a,BB,b,CC,c are sorted as : AA,BB,CC,a,b,c.

When false, they are sorted as : a,AA,b,BB,c,CC

@livecodepanos livecodepanos added this to the 9.5.1-rc-1 milestone Oct 9, 2019
@livecodepanos livecodepanos changed the base branch from develop-9.5 to develop October 9, 2019 08:43
@livecodepanos livecodepanos changed the base branch from develop to develop-9.5 October 9, 2019 08:43
@livecodepanos livecodepanos changed the base branch from develop-9.5 to develop October 9, 2019 09:30
@livecodepanos livecodepanos modified the milestones: 9.5.1-rc-1, 9.6.0-dp-1 Oct 9, 2019
@livecodeali livecodeali modified the milestones: 9.6.0-dp-1, 9.6.0-dp-2 Nov 27, 2019
@livecodepanos livecodepanos force-pushed the feature-treeview-sort-caseinsensitive branch from 821583f to 3522cd8 Compare November 29, 2019 08:06
end if
end repeat
end repeat
else
Copy link
Contributor

@livecodeian livecodeian Dec 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this intended to be used where there may be values that differ only in the case of their characters? (e.g. "Hello" vs "hello") If so, this method won't work correctly as it will replace both values with whichever appears first in the list

@livecodepanos livecodepanos removed this from the 9.6.0-dp-2 milestone Apr 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.