#380 patch for ID: 3136329

closed-rejected
None
5
2012-04-23
2011-02-01
Greg Knittl
No

This is the patch for ID: 3136329 no caret connecting to an existing file with -reuseview
The trunk EditServer has additional code to try and bring jEdit to the foreground just after
code this patch changes. On Ubuntu 10.04 this new code just flashes jEdit briefly but
doesn't bring jEdit to the foreground.

Conversely I hope that my patch doesn't affect Windows users and would encourage someone running
Windows to test this.

Discussion

  • Greg Knittl

    Greg Knittl - 2011-02-08

    I've updated the patch to remove the changes to EditServer. Explanation in ID: 3136329. This is now hopefully a low risk patch.

     
  • Alan Ezust

    Alan Ezust - 2011-08-29

    Shlomy please review?

     
  • Alan Ezust

    Alan Ezust - 2011-08-29
    • assigned_to: nobody --> shlomy
     
  • Alan Ezust

    Alan Ezust - 2011-08-30

    I am testing this patch and I can still see the caret disappears when I do a jedit -reuseview on debian with a file that is already open in jEdit, so i am not sure if it is complete yet.

     
  • Alan Ezust

    Alan Ezust - 2011-08-30
    • assigned_to: shlomy --> ezust
    • status: open --> pending
     
  • Alan Ezust

    Alan Ezust - 2011-08-30
    • status: pending --> pending-rejected
     
  • Alan Ezust

    Alan Ezust - 2011-08-30

    There is another issue that the jEdit window does not get raised to the top of the screen in reuseview from linux...
    That is quite annoying to me also.

    Your patch doesn't solve the issue described in 3136329, so I am not sure what it is supposed to do.
    I am marking it as rejected, but feel free to make another stab at it and I will review it.

     
  • Jarek Czekalski

    Jarek Czekalski - 2012-04-23
    • status: pending-rejected --> closed-rejected
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks