Magnify doesn't resize nicely

[atmartens]

If you make the magnify program small enough, the text runs too far outside the window, and the option button covers some of the text.

[atmartens]

The magnify program doesn't resize as it should

[negusnyul]

I attached a patch that automatically hides/shows the information text if needed.

Other solution: place B_AUTO_UPDATE_SIZE_LIMITS in the window constructor

however, thats not the best solution because the user cannot resize the window smaller than the default size - this is why I don't use it in the patch

[axeld]

Sorry for the very late reply; your patch has two issues that would need to be solved (I even tried, but the Magnify code is quite scary):

• When closing the window while the info is not visible due to the size, it will lose the setting.
• You make members public to get the preferred size of the widget. While that kind of fits to the coding style within Magnify, it would be much better to actually implement GetPreferredSize() for the info view instead, and use that (including the height of the window) to judge whether or not the info view should be visible.

[negusnyul]

Thanks for the reply, I will solve the mentioned issues eventually and create a new patch when I'll have the time.

[axeld]

Great, thanks! I didn't think you could be motivated to look into this again after that many month, it's appreciated, though!

[negusnyul]

It has been a long time but here is the new patch :)

(Created with git format-patch)

[mmadia]

Replying to negusnyul:

It has been a long time but here is the new patch :)

(Created with git format-patch)

Ping, for a review. comment:4 mentions the concerns with the first patch.

[korli]

Coding style issues:

• isInfoTextVisible() method name should be capitalized
• there is a space between the if and the parenthesis.
• braces for a single line if or else block should be dropped (but not in UpdateInfoBarOnResize()).
• the operator || shouldn't be last on a line.
• TInfoView::GetPreferredSize() declaration variable names don't match the implementation ones.

[mmadia]

Thanks for the patch negusnyul! Applied in hrev45301, along with korli's code review.