Removed unnecessary addition of 10 to browser height calculation and added comments for clarity.
This commit is contained in:
Kravickas 2026-03-30 02:51:34 +02:00 committed by GitHub
parent bc77c0cb42
commit 306edddf08
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -447,19 +447,19 @@ void update_manager::update(bool auto_accept)
changelog_browser->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
changelog_browser->setHtml(QStringLiteral("<h3>%0</h3>%1").arg(tr("Changelog:"), changelog_html));
// Measure height for 6 entries directly
// Measure height for 6 entries directly.
int browser_height;
if (changelog_count > 6)
{
// Temporarily render only 6 entries to get exact pixel height
changelog_browser->setHtml(QStringLiteral("<h3>%0</h3>%1").arg(tr("Changelog:"), changelog_html_capped));
browser_height = static_cast<int>(changelog_browser->document()->size().height()) + 10;
browser_height = static_cast<int>(changelog_browser->document()->size().height());
changelog_browser->setHtml(QStringLiteral("<h3>%0</h3>%1").arg(tr("Changelog:"), changelog_html));
}
else
{
browser_height = static_cast<int>(changelog_browser->document()->size().height()) + 10;
browser_height = static_cast<int>(changelog_browser->document()->size().height());
}
changelog_browser->setFixedSize(500, browser_height);
@ -472,6 +472,9 @@ void update_manager::update(bool auto_accept)
grid->addWidget(toggle_btn, row++, 0, 1, cols);
grid->addWidget(changelog_browser, row++, 0, 1, cols);
// Pre-size dialog so it doesn't resize when toggling
mb.setMinimumWidth(540);
QObject::connect(toggle_btn, &QPushButton::clicked, [changelog_browser, toggle_btn, &mb, show_text, hide_text]()
{
const bool becoming_visible = !changelog_browser->isVisible();