Modify action names

The names of the actions added to the link popup were not optimal. Fixed
that.

Also - I think `reinterpret_cast<>()` is too ... powerfull. We can use
`static_cast<>()` or even change the window type properly. Not important
now.

Author: diegoiast

Tab.cpp

@@ -78,19 +78,19 @@ Tab::Tab(QMainWindow* window)
         auto global_position = this->mapToGlobal(localPosition);
         auto menu = QMenu();
 
-        auto action1 = new QAction(tr("Copy Link "));
-        auto action2 = new QAction(tr("Open link in a new tab"));
+        auto copyLinkAction = new QAction(tr("Copy Link "));
+        auto openLinkInTanAction = new QAction(tr("Open link in a new tab"));
 
-        menu.addAction(action1);
-        menu.addAction(action2);
+        menu.addAction(copyLinkAction);
+        menu.addAction(openLinkInTanAction);
         auto res = menu.exec(global_position);
 
-        if (res == action1) {
+        if (res == copyLinkAction) {
             QClipboard *clipboard = QGuiApplication::clipboard();
             clipboard->setText(url.toString());
             qDebug() << "Copied to clipboard text:" << url.toString();
-        } else if (res == action2){
-            auto browser_window = reinterpret_cast<BrowserWindow*>(m_window);
+        } else if (res == openLinkInTanAction){
+            auto browser_window = static_cast<BrowserWindow*>(m_window);
             return browser_window->new_tab_with_url(url);
         }
     });