Skip to content

feat(i18n, layout): add dock orientation options and translations - #8

Merged
gitgusilva merged 1 commit into
gitgusilva:mainfrom
HermesSantos:feat/add-horizontal-changes-view
Jul 30, 2026
Merged

feat(i18n, layout): add dock orientation options and translations#8
gitgusilva merged 1 commit into
gitgusilva:mainfrom
HermesSantos:feat/add-horizontal-changes-view

Conversation

@HermesSantos

Copy link
Copy Markdown
Collaborator

Introduced new localization strings for "Dock to bottom" and "Dock to right" in English, Spanish, and Brazilian Portuguese. Updated layout service to manage details orientation, allowing users to toggle between bottom and right docking for the details panel in the History view.

Introduced new localization strings for "Dock to bottom" and "Dock to right" in English, Spanish, and Brazilian Portuguese. Updated layout service to manage details orientation, allowing users to toggle between bottom and right docking for the details panel in the History view.
@HermesSantos
HermesSantos requested a review from gitgusilva July 30, 2026 16:48
@HermesSantos

Copy link
Copy Markdown
Collaborator Author
image

@gitgusilva gitgusilva added enhancement New feature or request duplicate This issue or pull request already exists and removed duplicate This issue or pull request already exists labels Jul 30, 2026

@gitgusilva gitgusilva left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Approved. Pulled the branch locally, built the UI (vue-tsc -b && vite build clean) and ran it in dev: docking to the bottom, the resizer on the new axis and the persisted orientation all behave as expected. Reusing the existing Resizer with vertical/axis: 'y' and mirroring persistedSize in persistedString (with validation of the stored value) is the right fit for the codebase, and the default stays right, so nothing changes until the button is clicked.

Two notes, neither blocking. I will take care of them in a follow-up commit on main:

  • The panel container draws its border with border-neutral-200 dark:border-transparent. Since ef4fcfc every surface takes its border from the border-line token, so this one stops following the active theme. border-t border-line / border-l border-line keeps it consistent.
  • The dock button markup is repeated in the multi-select header and in #tab-actions, worth extracting if that block grows.

Thanks for the contribution.

@gitgusilva
gitgusilva merged commit 3b9f6de into gitgusilva:main Jul 30, 2026
2 checks passed
gitgusilva added a commit that referenced this pull request Jul 30, 2026
The dock-orientation change (#8) painted the panel's border with
border-neutral-200 dark:border-transparent, the pattern ef4fcfc removed
everywhere else. It leaves that edge outside the theme: a fixed grey under
every light theme, and no border at all under every dark one, whatever the
theme asked for. border-line is the token the other 258 borders in the app
already use.
gitgusilva added a commit that referenced this pull request Jul 30, 2026
GitBox moves from MIT to the GNU Lesser General Public License, version 3 or
later. Modifying GitBox itself now carries the obligation to publish those
modifications, which MIT never asked for, while linking the project into
another program stays permitted.

The LGPL is written as a set of additional permissions on top of the GPL and
carries no terms of its own, so both texts have to travel with the project:
LICENSE holds the LGPL, GPL-3.0.txt the GPL it refers to. The file it is
named LICENSE so the links in README and CONTRIBUTING keep working and so
GitHub still reports the licence; the second file is deliberately not named
COPYING, which would make the repository look dual-licensed to the detectors.

ui/electron/package.json gains the license field it never had. It is what
electron-builder writes into the deb and rpm metadata, so packages shipped
until now declared no licence at all.

Releases up to v1.2.0 went out under MIT and stay available under it: that
grant cannot be withdrawn, and nothing here tries to. The one outside
contribution merged so far (#8) came in under MIT, which permits
relicensing, so no consent is needed to carry it over.
gitgusilva added a commit that referenced this pull request Jul 31, 2026
The v1.3.0 entry was written from the last branch alone and left out the six
commits that landed before it — including the docked history detail panel
contributed in #8, which the pull request diff panel builds on, and the
relicensing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants