bugGNU Octave - Bugs: bug #46723, Resize close and dock butons on Mac

 
 

bug #46723: Resize close and dock butons on Mac

Submitted by:  Sebastian <sschoeps>
Submitted on:  Sat 19 Dec 2015 11:06:08 PM UTC  
 
Category: GUISeverity: 3 - Normal
Priority: 5 - NormalItem Group: Feature Request
Status: In ProgressAssigned to: None
Originator Name: SebastianOpen/Closed: Open
Release: devOperating System: Mac OS

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Sun 14 Aug 2016 11:08:41 AM UTC, comment #9:

Added a new patch "macosx_iconsize_qt5". I am unsure how device dependent this is. Maybe someone can check if this works on windows and linux, too?

(file #38206)

Sebastian <sschoeps>
Sun 14 Aug 2016 07:39:41 AM UTC, comment #8:

Octave compiles now with qt5. Can a qt expert help me to fix this?

Sebastian

Sebastian <sschoeps>
Sat 02 Jul 2016 01:57:18 AM UTC, comment #7:

Retagging "In progress".

Lachlan Andrew <lachlan>
Project Member
Tue 22 Dec 2015 02:41:06 PM UTC, comment #6:

For Qt5 refer to bug #40252. (No, the default branch does not compile with Qt5 yet).

Mike Miller <mtmiller>
Project Administrator
Tue 22 Dec 2015 10:15:22 AM UTC, comment #5:

Sebastian: Fedora 23 is at qt5. But ./configure
chooses qt4.

Michael Godfrey <godfrey>
Project Member
Tue 22 Dec 2015 09:58:11 AM UTC, comment #4:

Your patch seems to change nothing. See image attached.

I believe that we might need to rescale due to high resolution (on the mac it is called "retina"). In qt5 we could do rescaling with "devicePixelRatio" but this seems to be not available in qt4.

BTW: can octave be compiled with qt5?

(file #35834)

Sebastian <sschoeps>
Tue 22 Dec 2015 07:15:23 AM UTC, comment #3:

It seems that Qt uses a style with larger icons for the dock-widget compared to the main-window. The toolbar icons have correct size, the close/dock buttons not, both are based on Qt::PM_SmallIconSize from the related style. The larger style in the widgets are also reflected by the quite large folder icons in the file browser.

Could you please try the attached patch that uses the style from the main-window for the close/dock buttons of the widgets?

(file #35831)

Torsten <ttl>
Project Member
Mon 21 Dec 2015 10:25:51 AM UTC, comment #2:

The easy part: the screenshot that you were linking to uses icon size "normal". While the attached ones use size "small". However, this does not affect the close or undock buttons. I added a new attachment to make the difference clear.

The difficult part: my patch does not help. The problem of the huge buttons only occurs on the retina display (https://en.wikipedia.org/wiki/Retina_Display). If I attach an external screen with normal resolution to my Macbook and I open octave on that screen then everything is as expected. This explains the differences in the screenshots that you observed (the screenshot in my blog was made on a regular screen).

I guess nobody here has experience with issues like this?

(file #35811)

Sebastian <sschoeps>
Sun 20 Dec 2015 12:04:36 PM UTC, comment #1:

Thank you for the report and the patch. Feedback on the appearance of the gui on different systems is highly welcome.

Concerning the screenshot: Is your screenshot at
http://www.schoeps.org/home/wp-content/uploads/2015/12/octave41mac.png
with or without the patch? It seems to me that the dock/close icons have a size that differs from both sizes in the attached patch. Not related to this bug: Do you know what icon size was selected in the preferences for the screenshots?

Torsten <ttl>
Project Member
Sat 19 Dec 2015 11:06:08 PM UTC, original submission:

Probably the Mac GUI should be resigned by an expert, however, for now, please apply the attached patch to rescale the dock bar icons appropriately.

Sebastian <sschoeps>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #38206:  macosx_iconsize_qt5.patch added by sschoeps (881B - application/octet-stream - patch using qt5's devicePixelRatio)
file #35811:  icon_sizes.png added by sschoeps (113KiB - image/png)
file #35801:  Screenshots.png added by sschoeps (280KiB - image/png)
file #35802:  macosx_iconsize.patch added by sschoeps (637B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by lachlan (Posted a comment)
  • -unavailable- added by siko1056 (Updated the item)
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by godfrey (Posted a comment)
  • -unavailable- added by ttl (Posted a comment)
  • -unavailable- added by sschoeps (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 8 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 14 Aug 2016 11:08:41 AM UTCsschoepsAttached File-=>Added macosx_iconsize_qt5.patch, #38206
    Sat 02 Jul 2016 01:57:18 AM UTClachlanStatusPatch Submitted=>In Progress
    Mon 04 Jan 2016 10:28:44 PM UTCsiko1056StatusNone=>Patch Submitted
    Tue 22 Dec 2015 09:58:11 AM UTCsschoepsAttached File-=>Added patch_bug46723_main-win_style.png, #35834
    Tue 22 Dec 2015 07:15:23 AM UTCttlAttached File-=>Added bug46723_main-win_style.patch, #35831
    Mon 21 Dec 2015 10:25:51 AM UTCsschoepsAttached File-=>Added icon_sizes.png, #35811
    Sat 19 Dec 2015 11:06:08 PM UTCsschoepsAttached File-=>Added Screenshots.png, #35801
      Attached File-=>Added macosx_iconsize.patch, #35802

    Back to the top


    Powered by Savane 3.1-cleanup1