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

sort items in tree-browsers caseinsensitively in alphabetical order #5875

Merged

Conversation

frank-a-i
Copy link
Contributor

@frank-a-i frank-a-i commented Jan 4, 2021

This PR adjusts the item order alphabetically independent from naming style. Hence items will be sorted with the following manner:

  1. aa
  2. Ab
  3. Ba
  4. bb

Affects amongst others Instrument Plugins, My Projects, My Samples, My Presets, My Home, Root Directory and their sub items. See the issue for details

@LmmsBot
Copy link

LmmsBot commented Jan 4, 2021

🤖 Hey, I'm @LmmsBot from github.com/lmms/bot and I made downloads for this pull request, click me to make them magically appear! 🎩

Linux

Windows

macOS

🤖
{"platform_name_to_artifacts": {"Linux": [{"artifact": {"title": {"title": "(AppImage)", "platform_name": "Linux"}, "link": {"link": "https://12024-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.63%2Bg218d93b-linux-x86_64.AppImage"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12024?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}], "Windows": [{"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://12022-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.63%2Bg218d93bae-mingw-win32.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12022?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://12021-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.63%2Bg218d93bae-mingw-win64.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12021?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/iwtn3sxej0niww5b/artifacts/build/lmms-1.3.0-alpha-msvc2017-win32.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/37143547"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/799uusr4thpqg0cr/artifacts/build/lmms-1.3.0-alpha-msvc2017-win64.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/37143547"}], "macOS": [{"artifact": {"title": {"title": "", "platform_name": "macOS"}, "link": {"link": "https://12023-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.63%2Bg218d93bae-mac10.14.dmg"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12023?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}]}, "commit_sha": "21542189d0ed062ccf39a3c2c7580297868c9d54"}

@PhysSong PhysSong linked an issue Jan 5, 2021 that may be closed by this pull request
@tresf

This comment has been minimized.

@frank-a-i

This comment has been minimized.

src/gui/FileBrowser.cpp Outdated Show resolved Hide resolved
@tresf

This comment has been minimized.

src/gui/FileBrowser.cpp Outdated Show resolved Hide resolved
src/gui/FileBrowser.cpp Outdated Show resolved Hide resolved
@tresf
Copy link
Member

tresf commented Jan 6, 2021

Thanks! Just some whitespace formatting (hope you don't mind) and we'll merge!

src/gui/FileBrowser.cpp Outdated Show resolved Hide resolved
Copy link
Member

@Spekular Spekular left a comment

Choose a reason for hiding this comment

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

LGTM.

@Spekular Spekular merged commit f7fbb9a into LMMS:master Jan 7, 2021
@Spekular
Copy link
Member

Spekular commented Jan 7, 2021

Counting tresf's earlier comment as approval and adding my own => 2 approvals, merging. CI passed for previous commit (ignoring unrelated errors) and the only change since is an empty line.

Just some whitespace formatting (hope you don't mind) and we'll merge!

@frank-a-i frank-a-i deleted the 4925-ignore_capitals_in_token-search branch January 9, 2021 17:48
zynskeywolf pushed a commit to zynskeywolf/lmms that referenced this pull request Jan 12, 2021
* sort items in tree-browsers caseinsensitively in alphabetical order
IanCaio pushed a commit to IanCaio/lmms that referenced this pull request Mar 28, 2021
* sort items in tree-browsers caseinsensitively in alphabetical order
devnexen pushed a commit to devnexen/lmms that referenced this pull request Apr 10, 2021
* sort items in tree-browsers caseinsensitively in alphabetical order
sdasda7777 pushed a commit to sdasda7777/lmms that referenced this pull request Jun 28, 2022
* sort items in tree-browsers caseinsensitively in alphabetical order
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.

ignore capitals in Token-search
4 participants