D10650: Restore zoom'ed font sizes after changing profile properties via the command line or ESC sequences

[Kurt Hindenburg]

hindenburg added a comment.


  I'm testing using arc as well

INLINE COMMENTS

> SessionManager.cpp:271
> +
> +    foreach (TerminalDisplay *view, session->views()){
> +        if (view->getVTFont() != _sessionProfiles[session]->font()) {

const TerminaD...

> SessionManager.cpp:272
> +    foreach (TerminalDisplay *view, session->views()){
> +        if (view->getVTFont() != _sessionProfiles[session]->font()) {
> +            zoomFontSizes.insert(view, view->getVTFont());

since you are calling getVTfont() twice, you might want to assign it to a variable - depends on cost and how often called

REPOSITORY
  R319 Konsole

REVISION DETAIL
  https://phabricator.kde.org/D10650

To: ahmadsamir, hindenburg
Cc: ngraham, #konsole, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20180220/3d023d94/attachment.html>