Skip to content

Conversation

Utkarsh-khambra
Copy link
Collaborator

Fixes #1153, is this the correct way to solve this in QML world?

Yaraslaut
Yaraslaut previously approved these changes Aug 10, 2023
Copy link
Member

@Yaraslaut Yaraslaut left a comment

Choose a reason for hiding this comment

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

LGTM, tested it on ubuntu vm

Copy link
Member

@christianparpart christianparpart left a comment

Choose a reason for hiding this comment

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

LGTM. Many thanks!

@@ -297,7 +297,7 @@ void TerminalWidget::setSession(TerminalSession* newSession)

session_->start();

framelessWidget_ = !profile().show_title_bar;
window()->setFlag(Qt::FramelessWindowHint, !profile().show_title_bar);
Copy link
Member

Choose a reason for hiding this comment

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

good catch

@christianparpart christianparpart merged commit b53f223 into master Aug 11, 2023
@christianparpart christianparpart deleted the bugfix/1153 branch August 11, 2023 09:05
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.

show_title_bar options doesn't work as expected
3 participants