Code Monkey home page Code Monkey logo

Comments (10)

lukefromdc avatar lukefromdc commented on August 17, 2024

I wasn't able to duplicate this with visudo (based on vim) or with nano. I don't have vim itself installed as I prefer nano for such tasks

from mate-terminal.

mjd-tech avatar mjd-tech commented on August 17, 2024

You're right.
Not a problem with nano or less (the pager).
I've noticed it with: htop, fzf, mc (midnight commander), gdu, vim, neovim, dialog.
Maybe these use a different ncurses library or something. These support mouse, nano and less do not.
When I use Tilix instead of Mate-terminal, everything works as expected.
Mate-terminal was working fine until the "double-click tab to set title" feature was added.

from mate-terminal.

lukefromdc avatar lukefromdc commented on August 17, 2024

I don't have anything installed by default that triggers this. Presumably some of the other devs here do

from mate-terminal.

mjd-tech avatar mjd-tech commented on August 17, 2024

Thanks for looking at this.

Just to make sure this isn't a Manjaro/Arch problem, I did the following:

  • Installed Debian Testing with Mate Desktop, in a VM.
  • built mate-terminal 1.27.1 according to instructions.

Same issues as on Manjaro, so it's not a distro issue.

I took a look at xfce4-terminal's terminal-window.c
at line 1196, it looks like they are checking to make sure you actually double clicked on a tab.

Whereas in mate-terminal's terminal-window.c, line 2994 it just acts on a double click event.

Perhaps if we add xfce's check, it fixes the problem.
Unfortunately I don't know how to do this.

Here's the snippet from xfce4-terminal

        /* check if the user double-clicked on the label */
          label = gtk_notebook_get_tab_label (notebook, GTK_WIDGET (window->priv->active));
          if (terminal_window_notebook_event_in_allocation (x, y, label))
            {
              terminal_window_action_set_title (window);
              return TRUE;
            }
        }

from mate-terminal.

lukefromdc avatar lukefromdc commented on August 17, 2024

Looks like the general algorithm here is to ensure the click was in the tab space before opening the dialog

from mate-terminal.

lukefromdc avatar lukefromdc commented on August 17, 2024

Installed htop, issue confirmed here

from mate-terminal.

lukefromdc avatar lukefromdc commented on August 17, 2024

Try
#446

from mate-terminal.

mjd-tech avatar mjd-tech commented on August 17, 2024

I like your idea of just removing this feature.

The user can already set the title three ways:

  1. clicking "Terminal...Set Title" in the main menu bar.
  2. right-clicking on the tab and choose "Set Title"
  3. create a profile with the desired title

I think if mate-terminal is going to borrow an idea from xfce4-terminal,
it should be adding "Open file manager here" to the "File" menu.

from mate-terminal.

lukefromdc avatar lukefromdc commented on August 17, 2024

We have a better and simpler fix in #446 now, thanks to @cwendling
I will leave the choice betweeen reverting the prior offending commit and this fix to the rest of the team

from mate-terminal.

lukefromdc avatar lukefromdc commented on August 17, 2024

#446 is merged, so closing as the fix will automatically be in the next 1.27 point release

from mate-terminal.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.