bugGNU Octave - Bugs: bug #38265, OpenGL: changing legend fontsize...

 
 
Show feedback again

bug #38265: OpenGL: changing legend fontsize has no effect

Submitted by:  Ben Abbott <bpabbott>
Submitted on:  Tue 05 Feb 2013 02:13:26 PM UTC  
 
Category: Plotting with OpenGLSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: NoneAssigned to: Ben Abbott <bpabbott>
Originator Name: Ben AbbottOpen/Closed: Open
Release: devOperating System: Any

Add a New Comment (Rich MarkupRich 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)

Fri 17 May 2013 12:20:09 AM UTC, comment #8:

I'm leaving this assigned to me, but only to make it difficult for me for forget about it. If someone else is inclined to work on this, please go ahead.

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Wed 06 Feb 2013 05:26:09 PM UTC, comment #7:

I think I'm wrong in my comment below. The legend's box size is determined by the text objects extents in legend.m

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Wed 06 Feb 2013 05:12:14 PM UTC, comment #6:

I don't know where, but the updating the of axes box is done on the c++ side. Meanwhile there are no listeners on the c++ side.

Either the updater for the axes box needs to be changed to accommodate a change in fontsize, or we might try something kludgy (add a listener) in legend.m with a listener that triggers the updater on the c++ side to act.

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Wed 06 Feb 2013 04:39:29 PM UTC, comment #5:

Do we also need a listener that updates the box size when the fontsize changes? Using the first demonstration code the font size is correctly updated, but it now extends beyond the legend box. See the attached jpeg.

(file #27404)

Rik <rik5>
Project Member
Wed 06 Feb 2013 03:08:00 PM UTC, comment #4:

http://hg.savannah.gnu.org/hgweb/octave/rev/cddf9103a566

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Tue 05 Feb 2013 04:08:30 PM UTC, comment #3:

I've attached a changeset which addresses this bug and 38263

(file #27394)

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Tue 05 Feb 2013 03:29:37 PM UTC, comment #2:

I took a quick look at legend.m and found that the fix is easy. We had neglected to add a listener for the legend's fontsize property.

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Tue 05 Feb 2013 02:46:34 PM UTC, comment #1:

Changing the fontsize property on individual text objects works:

The opposite behaviour occurs with GNUPLOT (changing fontsize works on the legend object, but not on its text children).

Julien Bect <jbect>
Tue 05 Feb 2013 02:13:26 PM UTC, original submission:
Ben Abbott <bpabbott>
Project MemberIn charge of this item.

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #27404:  fontsize_tst.jpg added by rik5 (26kB - image/jpeg)
file #27394:  changeset.patch added by bpabbott (2kB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rik5 (Updated the item)
  • -unavailable- added by jbect (Posted a comment)
  • -unavailable- added by bpabbott (Submitted the item)
  •  

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

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 17 May 2013 12:20:09 AM UTCbpabbottStatusReady For Test=>None
    Wed 06 Feb 2013 04:39:29 PM UTCrik5Attached File-=>Added fontsize_tst.jpg, #27404
    Wed 06 Feb 2013 03:08:00 PM UTCbpabbottStatusPatch Submitted=>Ready For Test
    Tue 05 Feb 2013 04:08:49 PM UTCbpabbottStatusNone=>Patch Submitted
      Assigned toNone=>bpabbott
    Tue 05 Feb 2013 04:08:30 PM UTCbpabbottAttached File-=>Added changeset.patch, #27394
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup