Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Search
  • Get Qt Extensions
  • Unsolved
Collapse
Brand Logo
  1. Home
  2. Qt Development
  3. General and Desktop
  4. QLabel and right click
Forum Updated to NodeBB v4.3 + New Features

QLabel and right click

Scheduled Pinned Locked Moved Solved General and Desktop
18 Posts 4 Posters 1.5k Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • PerdrixP Perdrix

    Sorry I meant when I LEFT click, nothing happens

    David (I'll edit the original text to correct that)

    jsulmJ Offline
    jsulmJ Offline
    jsulm
    Lifetime Qt Champion
    wrote on last edited by
    #6

    @Perdrix It seems like you're relying on the auto-connect feature, right? I mean you do not connect the signal to slot explicitly. You could try to connect by yourself to see whether it makes a difference.

    https://forum.qt.io/topic/113070/qt-code-of-conduct

    1 Reply Last reply
    2
    • PerdrixP Offline
      PerdrixP Offline
      Perdrix
      wrote on last edited by
      #7

      Yes, I'm using auto-connect and I don't get any warnings about Qt being unable to connect the signals to the slots (as I did when I originally tried to use on_backgroundCalibration_clicked() - that definitely got a complaint.

      JonBJ jsulmJ 2 Replies Last reply
      0
      • PerdrixP Perdrix

        Yes, I'm using auto-connect and I don't get any warnings about Qt being unable to connect the signals to the slots (as I did when I originally tried to use on_backgroundCalibration_clicked() - that definitely got a complaint.

        JonBJ Offline
        JonBJ Offline
        JonB
        wrote on last edited by
        #8

        @Perdrix
        Please at least put a qDebug()/breakpoint inside your slot to verify being called, just in case it is being called and you (somehow) can't see the menu!

        1 Reply Last reply
        0
        • PerdrixP Perdrix

          Yes, I'm using auto-connect and I don't get any warnings about Qt being unable to connect the signals to the slots (as I did when I originally tried to use on_backgroundCalibration_clicked() - that definitely got a complaint.

          jsulmJ Offline
          jsulmJ Offline
          jsulm
          Lifetime Qt Champion
          wrote on last edited by
          #9

          @Perdrix The auto-connect feature is not that reliable. I would try with explicit connect. If it complains then post your connect call here and the error.

          https://forum.qt.io/topic/113070/qt-code-of-conduct

          1 Reply Last reply
          0
          • PerdrixP Offline
            PerdrixP Offline
            Perdrix
            wrote on last edited by Perdrix
            #10

            I added:

            connect(ui->backgroundCalibration, SIGNAL(linkActivated(const Qstring &)), this, SLOT(on_backgroundCalibration_linkActivated()));
            

            to the ctor code just after the ui->SetupUI() call.

            I have a breakpoint inside the slot code at the line that is intended to raise the menu, but it is never reached.

            David

            JonBJ 1 Reply Last reply
            0
            • PerdrixP Perdrix

              I added:

              connect(ui->backgroundCalibration, SIGNAL(linkActivated(const Qstring &)), this, SLOT(on_backgroundCalibration_linkActivated()));
              

              to the ctor code just after the ui->SetupUI() call.

              I have a breakpoint inside the slot code at the line that is intended to raise the menu, but it is never reached.

              David

              JonBJ Offline
              JonBJ Offline
              JonB
              wrote on last edited by
              #11

              @Perdrix
              If that is a paste of your code, Qstring is mis-spelt. You would get help if that is not connecting the slot if you changed to new-style signal-slot syntax, which would benefit you in the long run.

              1 Reply Last reply
              3
              • PerdrixP Perdrix

                I have a QLabel called backgroundCalibration whose text I have set to "<a>Some text</a>"

                I did:

                ui->backgroundCalibration->setVisible(true);
                ui->backgroundCalibration->setTextInteractionFlags(Qt::LinksAccessibleByMouse);
                ui->backgroundCalibration->setForegroundRole(QPalette::Link);
                
                void	StackingParameters::on_backgroundCalibration_linkActivated(const QString & str)
                {
                	str;
                	//
                	// Show the popup menu 
                	//
                	backgroundCalibrationMenu->exec(QCursor::pos());
                }
                
                

                When I display the Widget containing this label the text is displayed as "Some Text" and is displayed in Blue as I expected.

                However when I right-click left-click the text nothing happens! What crucial step did I miss?

                TIA
                David

                B Offline
                B Offline
                Bonnie
                wrote on last edited by Bonnie
                #12

                @Perdrix
                You cannot use "<a>Some text</a>" to expect a linkActivated signal.
                You must set some text as the link: "<a href="any text">Some text</a>", or the text won't even look like linked.

                JonBJ PerdrixP 2 Replies Last reply
                3
                • B Bonnie

                  @Perdrix
                  You cannot use "<a>Some text</a>" to expect a linkActivated signal.
                  You must set some text as the link: "<a href="any text">Some text</a>", or the text won't even look like linked.

                  JonBJ Offline
                  JonBJ Offline
                  JonB
                  wrote on last edited by
                  #13

                  @Bonnie
                  Good spot, as usual @Bonnie! Might be nice if the Qt docs reminded us of this somewhere, I didn't see much on what you had to do to a QLabel to make it a clickable link.

                  1 Reply Last reply
                  0
                  • PerdrixP Offline
                    PerdrixP Offline
                    Perdrix
                    wrote on last edited by
                    #14

                    Yes I spotted that just after I posted and fixed it.

                    So the code now reads:

                    connect(ui->backgroundCalibration, SIGNAL(linkActivated(const QString &)), this, SLOT(on_backgroundCalibration_linkActivated(const QString &)));
                    

                    and received no nastygrams about non-existent signals. Sadly the breakpoint is still not reached.

                    PS what's the new format for connecting the signals and slots? My book is strictly old school Qt4

                    David

                    jsulmJ 1 Reply Last reply
                    0
                    • PerdrixP Perdrix

                      Yes I spotted that just after I posted and fixed it.

                      So the code now reads:

                      connect(ui->backgroundCalibration, SIGNAL(linkActivated(const QString &)), this, SLOT(on_backgroundCalibration_linkActivated(const QString &)));
                      

                      and received no nastygrams about non-existent signals. Sadly the breakpoint is still not reached.

                      PS what's the new format for connecting the signals and slots? My book is strictly old school Qt4

                      David

                      jsulmJ Offline
                      jsulmJ Offline
                      jsulm
                      Lifetime Qt Champion
                      wrote on last edited by
                      #15

                      @Perdrix Did you read what @Bonnie wrote?
                      https://wiki.qt.io/New_Signal_Slot_Syntax/de

                      https://forum.qt.io/topic/113070/qt-code-of-conduct

                      1 Reply Last reply
                      0
                      • B Bonnie

                        @Perdrix
                        You cannot use "<a>Some text</a>" to expect a linkActivated signal.
                        You must set some text as the link: "<a href="any text">Some text</a>", or the text won't even look like linked.

                        PerdrixP Offline
                        PerdrixP Offline
                        Perdrix
                        wrote on last edited by
                        #16

                        @Bonnie Silly question - why can't I just use <a>Text</a>? It is after all a valid minimal HTML anchor?

                        I shall change the code immediately to use the format you suggested.

                        D.

                        JonBJ 1 Reply Last reply
                        0
                        • PerdrixP Perdrix

                          @Bonnie Silly question - why can't I just use <a>Text</a>? It is after all a valid minimal HTML anchor?

                          I shall change the code immediately to use the format you suggested.

                          D.

                          JonBJ Offline
                          JonBJ Offline
                          JonB
                          wrote on last edited by JonB
                          #17

                          @Perdrix
                          Because <a>Text</a> is not clickable in HTML, it just defines an anchor point user can jump to (via # on URL), see e.g. https://stackoverflow.com/questions/10510191/valid-to-use-a-anchor-tag-without-href-attribute.

                          1 Reply Last reply
                          1
                          • PerdrixP Offline
                            PerdrixP Offline
                            Perdrix
                            wrote on last edited by
                            #18

                            @Bonnie said in QLabel and right click:

                            href="any text"

                            Thanks Bonnie - that solved it ...

                            1 Reply Last reply
                            0

                            • Login

                            • Login or register to search.
                            • First post
                              Last post
                            0
                            • Categories
                            • Recent
                            • Tags
                            • Popular
                            • Users
                            • Groups
                            • Search
                            • Get Qt Extensions
                            • Unsolved