bugGNU Octave - Bugs: bug #48696, press space on plot window makes...

 
 

bug #48696: press space on plot window makes pop-up hidden console on GUI mode

Submitted by:  Tatsuro MATSUOKA <tmacchant>
Submitted on:  Thu 04 Aug 2016 04:16:29 AM UTC  
 
Category: Plotting with gnuplotSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: Tatsuro MATSUOKAOpen/Closed: Closed
Release: devOperating System: Microsoft Windows

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)

Thu 25 Aug 2016 04:29:43 AM UTC, comment #8:

Closing report

Avinoam Kalma <avinoam>
Project Member
Thu 25 Aug 2016 02:21:20 AM UTC, comment #7:

I have confirmed the fix.
Thanks.
Please close this tracker.

Tatsuro MATSUOKA <tmacchant>
Tue 23 Aug 2016 09:23:06 PM UTC, comment #6:

Pushed http://hg.octave.org/mxe-octave/rev/851b9f4a6b1c

John Donoghue <lostbard>
Project Member
Tue 09 Aug 2016 05:22:42 AM UTC, comment #5:

I have made a changeset to compile gnuplot for windows with flag "DISABLE_SPACE_RAISES_CONSOLE".

Attachment file : bug48696-01.patch

This patch disable "space raise console for gnuplot windows"
and accidental pop-up of console can be avoided as long as users use
gnuplot embedded in octave for windows.

John Donoghue
Please verify this patch, modify it (if necessary) and apply it to the source tree.

Mike wrote in comment #1

> I assume the gnuplot installer from gnuplot.info would have this feature enabled, so if someone chooses to use a different gnuplot, this will still be present.


Perhaps the motivation to use gnuplot on the official site instead of embedded one is demand to use full feature of gnuplot graphics_toolkit.
Print command in gnuplot graphics_toolkit supports cairo based ones tikz ones. If gnuplot embedded with octave supports those features
, most user use it.
However this discussion should be made in bug #48474.

(file #38153)

Tatsuro MATSUOKA <tmacchant>
Fri 05 Aug 2016 05:37:06 AM UTC, comment #4:

I saw the gnuplot code online and found that what I have to do.

Add -DDISABLE_SPACE_RAISES_CONSOLE flag for compile.

Tatsuro MATSUOKA <tmacchant>
Fri 05 Aug 2016 05:17:58 AM UTC, comment #3:

I have consulted the gnuplot manual by my smart phone. The bind command cannot redefine the space key as exception. I have to see gnuplot code. After trip I will comit it asap unless someone else do it.

Tatsuro MATSUOKA <tmacchant>
Fri 05 Aug 2016 12:56:56 AM UTC, comment #2:

For build of gnuplot for windows it does not use configure. One has to edit Makefile and use it for build.
In the Makefile, no option that related to disable-raise-console exists.
Thefore one should attach a patch to the source code to disable this feature.
I may provide a patch for gnuplot in mxe build.

As you described, gnuplot for windows on the official site have this feature. So that there is a possibilty the phenomenon occurs, if users use external gnuplot.

BTW, gnuplot has "bind" command which bind key press on plot window to gnuplot actions. There is possibility to handle this issue on octave side.

I will do a short trip and I will commit again after that.

Tatsuro MATSUOKA <tmacchant>
Thu 04 Aug 2016 07:27:56 PM UTC, comment #1:

I think the only thing that could be done about this would be to build gnuplot with the configure option --disable-raise-console to turn off the feature completely.

Unless someone knows of a way to somehow "register" Octave as the console window that gnuplot should be raising.

I assume the gnuplot installer from gnuplot.info would have this feature enabled, so if someone chooses to use a different gnuplot, this will still be present.

Not really an Octave bug, unless you think we should disable this feature completely for mxe builds.

Mike Miller <mtmiller>
Project Administrator
Thu 04 Aug 2016 04:16:29 AM UTC, original submission:

OS : Microsoft windows
Version : 4.0 and dev
octave : GUI
graphics_toolkit : gnuplot
gnuplot terminal : windows, wxt

On plot window, press space key
=> pop-up hidden console!

See an attachment file "popup_console.png"

Note
1. The gnuplot has a feature some keys press on plot window execute gnuplot actions.

Press 'h' key on plot window displays the key and mouse binding on console.

According to the above, space key is binding to

Perhaps this gnuplot action makes pop-up hidden console on windows

2. On linux, noting happen by pressing space key on plot window.

3. On qt terminal, press space key does not work on all platform so that causes any problem.

4. On cli mode press space key raise octave command console. This is the same as that on the linux.

Tatsuro MATSUOKA <tmacchant>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #38153:  bug48696-01.patch added by tmacchant (2KiB - text/x-patch)
file #38099:  popup_console.png added by tmacchant (15KiB - image/png)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by avinoam (Posted a comment)
  • -unavailable- added by lostbard (Posted a comment)
  • -unavailable- added by tmacchant
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by tmacchant (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 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 25 Aug 2016 04:29:43 AM UTCavinoamStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Tue 23 Aug 2016 09:23:06 PM UTClostbardStatusNeed Info=>Ready For Test
    Tue 09 Aug 2016 05:22:42 AM UTCtmacchantCarbon-Copy-=>Added -unavailable-
    Tue 09 Aug 2016 05:22:41 AM UTCtmacchantAttached File-=>Added bug48696-01.patch, #38153
    Thu 04 Aug 2016 07:27:56 PM UTCmtmillerStatusNone=>Need Info
    Thu 04 Aug 2016 04:16:29 AM UTCtmacchantAttached File-=>Added popup_console.png, #38099

    Back to the top


    Powered by Savane 3.1-cleanup1