Skip to content

fix: окно вмещает интерфейс без прокрутки - #51

Open
bivlked wants to merge 1 commit into
mainfrom
fix/window-size
Open

bivlked wants to merge 1 commit into
mainfrom
fix/window-size

Conversation

@bivlked

@bivlked bivlked commented Oct 25, 2026

Copy link
Copy Markdown
Owner

После конвертации на выходной день с референсом в окне появлялась вертикальная прокрутка: результат не помещался.

Причина

Width и Height в Wails задают внешний размер окна вместе с рамкой и заголовком. При 360x758 самой странице доставалось около 345x720 CSS px (замерил GetClientRect при масштабе 150%). А интерфейсу в худшем случае нужно 361x782: месяц в 6 недель, выходной день, большая сумма и строка сравнения с референсом.

Что изменил

  • Окно 376x822, внутри около 361x784. Вёрстку не трогал.
  • Комментарий к --app-width в CSS, размер окна в руководстве пользователя, запись в CHANGELOG (Unreleased). Туда же добавил обновление зависимостей из chore(deps): bump github.com/labstack/echo/v4 from 4.15.3 to 4.16.1 #49.

Проверка

  • Собранный exe: окно 376x822, внутри 361,3x784,7, прокрутки нет.
  • Высоту страницы мерил в Edge при 361x784. Случай со скриншота (12.10.2026, референс 25.10.2026, 638 $) занимает 744 px. Август 2026 с 09.08, суммой 123 456 789,12 и референсом, в USD и в EUR, занимает 782 px.
  • Суммы до 999 999 999 999,99 помещаются. Прокрутка появляется только с 10 триллионов, такие суммы я оставил как есть.
  • golangci-lint без замечаний, go test ./... и тесты frontend (21/21) проходят.

Релиз пока не делаю, в эту ветку будут ещё доработки.

Width и Height в Wails задают внешний размер окна вместе с рамкой и
заголовком. При 360x758 странице оставалось около 345x720, и после
конвертации на выходной с референсом появлялась прокрутка. Месяцу в
6 недель с большой суммой нужно 361x782.

Окно теперь 376x822, внутри около 361x784. CHANGELOG и руководство
пользователя обновлены.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-25T06:52:23.384470Z 78b08ea PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 78b08eacc6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread main_gui.go
// около 361x784 - столько нужно странице в худшем случае (месяц в 6 недель,
// выходной, большая сумма и сравнение с референсом)
Width: 376,
Height: 822,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep the fixed window within supported displays

On the documented minimum 1280×720 display (docs/01-ТЕХНИЧЕСКОЕ-ЗАДАНИЕ.md:196 and the user guide at line 69), an 822 px outer window cannot fit even before accounting for the Windows taskbar. Since MinHeight, MaxHeight, and DisableResize also lock this dimension, the lower controls and conversion result can remain outside the visible screen with no way to resize the window; size the window against the available work area or preserve scrolling on displays shorter than the preferred layout.

Useful? React with 👍 / 👎.

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.

1 participant