Set the size of the main window to 600p of width and 500p of height when the application starts. #10

Merged
MicheleC merged 1 commits from fix/mainWindowSize into master 3 years ago
Ghost commented 3 years ago

@MicheleC I don't know if It's the right way to do It but when the application starts, the main window is too small.
Darrell would have called this "a paper cut", I think.

To test, delete the saved config if you had started kaffeine before (rm ~/.trinity/share/config/kaffeinerc).

@MicheleC I don't know if It's the right way to do It but when the application starts, the main window is too small. Darrell would have called this "a paper cut", I think. To test, delete the saved config if you had started kaffeine before (rm ~/.trinity/share/config/kaffeinerc).
Ghost added 1 commit 3 years ago
Ghost commented 3 years ago
Poster

here is original size when the application starts.

here is original size when the application starts.
Ghost commented 3 years ago
Poster

here is sized 600x500

here is sized 600x500
MicheleC requested changes 3 years ago
MicheleC left a comment
Owner

Nice catch Greg. It looks good, just need to fix the indentation and then I will merge.

Nice catch Greg. It looks good, just need to fix the indentation and then I will merge.
setupActions();
setStandardToolBarMenuEnabled(true);
//createStandardStatusBarAction();
resize( TQSize(600, 500) );
Owner

Could you adjsut the indentation? The rest of the code use a single tabm while this line uses 4 spaces, so it looks misaligned when tab size != 4.

Could you adjsut the indentation? The rest of the code use a single tabm while this line uses 4 spaces, so it looks misaligned when tab size != 4.
Owner

This is used only at first startup? This does not override the last saved window size?

This is used only at first startup? This does not override the last saved window size?
Owner

yes, only first start up without configuration. I can replicate it the same way.

yes, only first start up without configuration. I can replicate it the same way.
MicheleC marked this conversation as resolved
Ghost force-pushed fix/mainWindowSize from e2a390feaa to 6658071a08 3 years ago
MicheleC merged commit 6658071a08 into master 3 years ago
MicheleC deleted branch fix/mainWindowSize 3 years ago
MicheleC added this to the R14.0.10 release milestone 3 years ago
MicheleC approved these changes 3 years ago
Owner

Thanks Greg!

Thanks Greg!

Reviewers

MicheleC approved these changes 3 years ago
The pull request has been merged as 6658071a08.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/kaffeine#10
Loading…
There is no content yet.