GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Feature/sort cycle by deepsidhu1313 · Pull Request #4239 · sqlitebrowser/sqlitebrowser

Conversation

@deepsidhu1313

@deepsidhu1313 deepsidhu1313 commented Oct 31, 2026

Copy link
Copy Markdown
Contributor
  • Add option to cycle column header sorting through ascending, descending and none

  • Clicking the header of a sorted column only toggled between ascending and descending order, so removing the sorting required the separate 'Clear sorting' action. A new setting in the Data Browser preferences makes a click on a descending column remove the sorting instead, so repeated clicks cycle through ascending, descending and no sorting. This also works for Ctrl+click, where only the clicked column is removed from the sort columns.

The default behaviour is unchanged.
The decision how a click changes the sort columns moves from TableBrowser::headerClicked into sqlb::updateSortColumns so that it can be covered by a unit test.

  • New Record (Ctrl+Shift+N), Delete Record (Ctrl+Shift+D) and Revert Changes (Ctrl+Shift+R) had no keyboard shortcut. The two record actions are limited to the table browser which has the focus, so they do not clash when there are multiple browsers. The shortcuts are also added to the tool tips.
    A test checks that the actions have shortcuts which are unique among the actions and do not shadow standard keys.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
claude added 2 commits October 26, 2026 11:58
@claude
Add option to cycle column header sorting through ascending, descendi…
0ae2df7
…ng and none

Clicking the header of a sorted column only toggled between ascending and
descending order, so removing the sorting required the separate 'Clear
sorting' action. A new setting in the Data Browser preferences makes a
click on a descending column remove the sorting instead, so repeated clicks
cycle through ascending, descending and no sorting. This also works for
Ctrl+click, where only the clicked column is removed from the sort columns.
The default behaviour is unchanged.

The decision how a click changes the sort columns moves from
TableBrowser::headerClicked into sqlb::updateSortColumns so that it can be
covered by a unit test.
@claude
Add shortcuts for New Record, Delete Record and Revert Changes
f93c8e9
New Record (Ctrl+Shift+N), Delete Record (Ctrl+Shift+D) and Revert Changes
(Ctrl+Shift+R) had no keyboard shortcut. The two record actions are limited
to the table browser which has the focus, so they do not clash when there are
multiple browsers. The shortcuts are also added to the tool tips.

A test checks that the actions have shortcuts which are unique among the
actions and do not shadow standard keys.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@deepsidhu1313 @claude
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.