#3459 navigationwidget: allow renaming of note headings

Signed-off-by: Patrizio Bekerle <patrizio@bekerle.com>

Author: pbek

CHANGELOG.md

@@ -1,5 +1,13 @@
 # QOwnNotes Changelog
 
+## 26.2.1
+
+- Added ability to rename headings directly in the Navigation panel
+  (for [#3459](https://github.com/pbek/QOwnNotes/issues/3459))
+  - You can now double-click or press `F2` on any heading in the "Headings" tab of the Navigation panel to rename it
+  - The heading text in your note will update automatically while preserving the heading level
+  - This provides a quick way to reorganize your note structure without manually editing the text
+
 ## 26.2.0
 
 - Empty note files are now allowed again and will only result in a warning in the log

src/mainwindow.cpp

@@ -433,6 +433,9 @@ MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent) {
     // act on position clicks in the navigation widget
     connect(ui->navigationWidget, &NavigationWidget::positionClicked, this,
             &MainWindow::onNavigationWidgetPositionClicked);
+    // act on heading renames in the navigation widget
+    connect(ui->navigationWidget, &NavigationWidget::headingRenamed, this,
+            &MainWindow::onNavigationWidgetHeadingRenamed);
     connect(ui->backlinkWidget, &BacklinkWidget::noteClicked, this,
             &MainWindow::onBacklinkWidgetNoteClicked);
 
@@ -9630,6 +9633,48 @@ void MainWindow::onNavigationWidgetPositionClicked(int position) {
     ui->navigationWidget->setFocus();
 }
 
+/**
+ * Handles renaming of headings from the navigation widget
+ * Updates the heading text in the note
+ */
+void MainWindow::onNavigationWidgetHeadingRenamed(int position, const QString &oldText,
+                                                  const QString &newText) {
+    Q_UNUSED(oldText)
+
+    QOwnNotesMarkdownTextEdit *textEdit = activeNoteTextEdit();
+    QTextDocument *doc = textEdit->document();
+
+    // Find the block at the given position
+    QTextBlock block = doc->findBlock(position);
+    if (!block.isValid()) {
+        return;
+    }
+
+    // Get the current text of the block
+    QString blockText = block.text();
+
+    // Extract the heading level (number of # symbols)
+    static const QRegularExpression headingRegex(QStringLiteral("^(#+)\\s+"));
+    QRegularExpressionMatch match = headingRegex.match(blockText);
+
+    if (!match.hasMatch()) {
+        return;
+    }
+
+    QString headingPrefix = match.captured(1);    // The "###" part
+    QString newBlockText = headingPrefix + " " + newText;
+
+    // Update the text in the document
+    // We need to be careful to only replace the text content, not the block delimiters
+    QTextCursor cursor(block);
+    cursor.movePosition(QTextCursor::StartOfBlock);
+    cursor.movePosition(QTextCursor::EndOfBlock, QTextCursor::KeepAnchor);
+    cursor.insertText(newBlockText);
+
+    // Reparse the navigation to reflect the change
+    // This will be triggered automatically by the text change event
+}
+
 /**
  * Jumps to the note that was clicked in the backlink widget
  */

src/mainwindow.h

@@ -272,6 +272,9 @@ class MainWindow : public QMainWindow {
 
     void onNavigationWidgetPositionClicked(int position);
 
+    void onNavigationWidgetHeadingRenamed(int position, const QString &oldText,
+                                          const QString &newText);
+
    private slots:
 
     void on_noteTextEdit_textChanged();

src/widgets/navigationwidget.cpp

@@ -22,11 +22,13 @@
 #include <QTreeWidgetItem>
 
 NavigationWidget::NavigationWidget(QWidget *parent) : QTreeWidget(parent) {
-    // we want to handle currentItemChanged because it also works with the keyboard
+    // We want to handle currentItemChanged because it also works with the keyboard
     QObject::connect(this, &NavigationWidget::currentItemChanged, this,
                      &NavigationWidget::onCurrentItemChanged);
-    // we want to handle itemClicked because it allows to click on an item a 2nd time
+    // We want to handle itemClicked because it allows to click on an item a 2nd time
     QObject::connect(this, &NavigationWidget::itemClicked, this, &NavigationWidget::onItemClicked);
+    // We want to handle itemChanged to allow renaming headings
+    QObject::connect(this, &NavigationWidget::itemChanged, this, &NavigationWidget::onItemChanged);
 }
 
 /**
@@ -57,6 +59,35 @@ void NavigationWidget::onItemClicked(QTreeWidgetItem *current, int column) {
     emit positionClicked(current->data(0, Qt::UserRole).toInt());
 }
 
+/**
+ * Handles renaming of heading items
+ * Emits the headingRenamed signal when a heading is renamed
+ */
+void NavigationWidget::onItemChanged(QTreeWidgetItem *item, int column) {
+    Q_UNUSED(column)
+
+    if (item == nullptr) {
+        return;
+    }
+
+    // Get the position from the item
+    int position = item->data(0, Qt::UserRole).toInt();
+
+    // Find the corresponding node in our cache
+    for (const auto &node : _navigationTreeNodes) {
+        if (node.pos == position) {
+            QString oldText = node.text;
+            QString newText = item->text(0);
+
+            // Only emit if the text actually changed
+            if (oldText != newText && !newText.isEmpty()) {
+                emit headingRenamed(position, oldText, newText);
+            }
+            break;
+        }
+    }
+}
+
 /**
  * Parses a text document and builds the navigation tree for it
  */
@@ -164,6 +195,8 @@ void NavigationWidget::buildNavTree(const QVector<Node> &nodes) {
         item->setText(0, stripMarkdown(node.text));
         item->setData(0, Qt::UserRole, pos);
         item->setToolTip(0, tr("headline %1").arg(elementType - MarkdownHighlighter::H1 + 1));
+        // Make the item editable to allow renaming headings
+        item->setFlags(item->flags() | Qt::ItemIsEditable);
 
         // attempt to find a suitable parent item for the element type
         QTreeWidgetItem *lastHigherItem = findSuitableParentItem(elementType);

src/widgets/navigationwidget.h

@@ -43,13 +43,15 @@ class NavigationWidget : public QTreeWidget {
    private slots:
     void onCurrentItemChanged(QTreeWidgetItem *current, QTreeWidgetItem *previous);
     void onItemClicked(QTreeWidgetItem *current, int column);
+    void onItemChanged(QTreeWidgetItem *item, int column);
 
    private:
     void buildNavTree(const QVector<Node> &nodes);
     void doParse();
 
    signals:
     void positionClicked(int position);
+    void headingRenamed(int position, const QString &oldText, const QString &newText);
 
    private:
     const QTextDocument *_doc;