Thread: Some ideas
View Single Post
Old 03-15-2024, 12:29 PM   #7
KevinH
Sigil Developer
KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.
 
Posts: 7,734
Karma: 5446592
Join Date: Nov 2009
Device: many
Thanks. The space after the comma needs to be controlled by the formatting settings to be fully correct. Typically it is not added for single line formats.

Add adding highlighting for changed or unchanged is doable but since this is Regex Rename dialog, it should probably only highlight unchanged. Or as you said, if you do not want to rename a file do not include it when selecting the set of files for Regex rename.



Quote:
Originally Posted by BeckyEbook View Post
Ad 1.
a) The number of items displayed in the table corresponds to the number of files selected before Rename RegEx was selected. Personally, I only select the files whose names I want to change....
b) I think it is possible to reconcile both sides.
Changed elements can be highlighted in the table with a background colour, for example. (Below is an example, but you have to take light/dark mode into account).

Spoiler:
Code:
    foreach(QString bpath, bookpaths) {
        QString apath = newbookpaths.at(r);
        ui.tableWidget->setItem(r, 0, new QTableWidgetItem(bpath));
        ui.tableWidget->setItem(r, 1, new QTableWidgetItem(apath));
        if (apath != bpath) {
            ui.tableWidget->item(r, 1)->setBackground(Qt::darkRed);
        }
        r++;
    }


Ad 2.
This line (add space after comma).

Although I have checked this, it is worth investigating further.
KevinH is offline   Reply With Quote