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

 
 

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: FixedAssigned to: Ben Abbott <bpabbott>
Originator Name: Ben AbbottOpen/Closed: Closed
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)

Thu 06 Jun 2013 04:19:12 PM UTC, comment #10:

Works for me now. Thanks Ben. Closing report.

Rik <rik5>
Project Administrator
Thu 06 Jun 2013 05:16:59 AM UTC, comment #9:

I've pushed a changeset that works for me.

http://hg.savannah.gnu.org/hgweb/octave/rev/5cf7369a74cb

To test, run the first demo.

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
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 Administrator
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)
  •  

    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 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 06 Jun 2013 04:19:12 PM UTCrik5StatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Thu 06 Jun 2013 05:16:59 AM UTCbpabbottStatusNone=>Ready For Test
    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

    Back to the top


    Powered by Savane 3.1-cleanup