Skip to content

Inverting the selection in multi-profile selection (#140) - #144

Open
Fasdeq13 wants to merge 8 commits into
throneproj:mainfrom
Fasdeq13:invert-profile-selection
Open

Fasdeq13 wants to merge 8 commits into
throneproj:mainfrom
Fasdeq13:invert-profile-selection

Conversation

@Fasdeq13

Copy link
Copy Markdown
Contributor

I looked into #140, which requested an action for multi-selecting profiles. Options like testing, deleting, QR, clearing, and "select all" were already present; all that was missing was an "invert selection" feature. I’ve added it.

Changes made:

  • SelectionMode.kt: Added an invertSelection() function. It iterates through the visible profiles in the list and toggles each checkbox. If no items remain selected after this operation, the selection mode closes—just as it would when the last checkbox is unchecked. I also hooked up the new menu item in onMenuItemClick.
  • Profile_selection_menu.xml: Added the action_selection_invert item to the selection menu, positioned right after "Select all."
  • values/strings.xml: Added the string resource Profiles_invert_selection ("Invert selection").
  • values-ru/strings.xml: Added the corresponding Russian translation ("Инвертировать выбор").

I haven’t touched the other languages; they will default to English.

Closes #140

@Jaanyaar

Copy link
Copy Markdown

Some actions not available on multiselecting:
resolve domain
clear statisticks

@Fasdeq13

Copy link
Copy Markdown
Contributor Author

Some actions not available on multiselecting: resolve domain clear statisticks

Done, added Resolve domains and Clear traffic statistics to the multiselect menu. Updated the PR.

@Jaanyaar

Jaanyaar commented Nov 15, 2026 •

Copy link
Copy Markdown

Some actions not available on multiselecting: resolve domain clear statisticks

Done, added Resolve domains and Clear traffic statistics to the multiselect menu. Updated the PR.

Thank you. Sharing needs enhancements for consistency:
On mutiselecting, no configuration available

Names are not the same: copy link vs export to clipboard

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.

[UI/Theme]

2 participants