0.4.11: slider margini rimossi dalle Impostazioni (resta il dialog nel lettore); margini alto/basso strutturali — finestra di lettura dentro i margini, identici su ogni pagina

Lettore 0.4.10: sfondo di lettura uniforme (colore pieno per tema) — le fasce margini usano lo stesso colore: margine su ogni pagina senza bande ne' testo coperto percepibile
Lettore 0.4.9: margini verticali su ogni schermata — fasce opache fisse sopra/sotto la viewport (colore di sfondo per tema) + padding deficit dell'ultima pagina (max-scroll multiplo esatto del passo: ultima pagina allineata, niente righe ripetute)
2026-09-07 17:47:57 +02:00 · 2026-09-07 17:39:28 +02:00 · 2026-09-07 17:31:07 +02:00 · 2026-09-03 14:16:18 +02:00
18 changed files with 337 additions and 174 deletions
@@ -10,14 +10,16 @@ SOURCES += \
    src/apiclient.cpp \
    src/downloader.cpp \
    src/opdsparser.cpp \
-    src/epub.cpp
+    src/epub.cpp \
+    src/readerstate.cpp

 HEADERS += \
    src/settings.h \
    src/apiclient.h \
    src/downloader.h \
    src/opdsparser.h \
-    src/epub.h
+    src/epub.h \
+    src/readerstate.h

 LIBS += -lz

@@ -33,8 +35,9 @@ DISTFILES += $$OTHER_FILES

 # Translations are NOT handled by the sailfishapp_i18n template on the 5.1 kit
 # (the .qm is never generated/installed reliably). The spec builds the .qm
-# with lrelease and installs it into /usr/share/translations directly;
-# libsailfishapp loads harbour-calibreweb_<locale>.qm automatically.
+# with lrelease and installs it into the app data dir
+# (/usr/share/harbour-calibreweb/translations — the RPM validator rejects
+# /usr/share/translations); main.cpp loads it explicitly (see language block).
 # Source strings are ENGLISH (default language); harbour-calibreweb_it.ts
 # provides the Italian catalogue.

@@ -1,10 +1,14 @@
 import QtQuick 2.0
 import Sailfish.Silica 1.0

+// Cover: se un libro è aperto (readerState attivo dal ReaderPage) mostra
+// titolo e posizione di lettura, altrimenti icona e nome dell'app.
 CoverBackground {
    Column {
        anchors.centerIn: parent
+        width: parent.width - 2 * Theme.paddingLarge
        spacing: Theme.paddingMedium
+        visible: !readerState.active

        Image {
            anchors.horizontalCenter: parent.horizontalCenter
@@ -20,4 +24,29 @@ CoverBackground {
            font.pixelSize: Theme.fontSizeSmall
        }
    }
+
+    Column {
+        anchors.centerIn: parent
+        width: parent.width - 2 * Theme.paddingLarge
+        spacing: Theme.paddingSmall
+        visible: readerState.active
+
+        Label {
+            width: parent.width
+            text: readerState.title
+            color: Theme.primaryColor
+            font.pixelSize: Theme.fontSizeSmall
+            horizontalAlignment: Text.AlignHCenter
+            truncationMode: TruncationMode.Elide
+        }
+
+        Label {
+            width: parent.width
+            text: readerState.subtitle
+            color: Theme.secondaryColor
+            font.pixelSize: Theme.fontSizeExtraSmall
+            horizontalAlignment: Text.AlignHCenter
+            truncationMode: TruncationMode.Elide
+        }
+    }
 }
@@ -9,6 +9,7 @@ ApplicationWindow {
    initialPage: Component { MainPage {} }
    cover: Component { CoverPage {} }

-    allowedOrientations: Orientation.Portrait
-    _defaultPageOrientations: Orientation.Portrait
+    // Niente vincoli a livello window: ogni pagina dichiara le proprie
+    // orientazioni (il lettore ruota in landscape, le pagine di lista
+    // restano in portrait). _defaultPageOrientations è API privata.
 }
@@ -4,6 +4,8 @@ import Sailfish.Silica 1.0
 Page {
    id: page

+    allowedOrientations: Orientation.Portrait
+
    property var bookData: ({})
    property string coverSource: ""
    property bool downloading: false
@@ -4,6 +4,8 @@ import Sailfish.Silica 1.0
 Page {
    id: page

+    allowedOrientations: Orientation.Portrait
+
    property string feedUrl
    property string pageTitle
    property string nextUrl: ""
@@ -4,6 +4,8 @@ import Sailfish.Silica 1.0
 Page {
    id: page

+    allowedOrientations: Orientation.Portrait
+
    property bool loading: false
    property string errorMessage: ""

@@ -18,7 +18,6 @@ Page {
    property string bookFormat: "epub"

    property int currentChapter: 0
-    property double chapterFraction: 0.0
    // posizione di lettura nel capitolo corrente, stimata in caratteri di testo
    // dalla frazione di scroll (stabile col reflow del font)
    property int charOffset: 0
@@ -143,10 +142,18 @@ Page {
            }
        }

-        // contenuto: Flickable NON interattivo (lo scroll è solo programmatico)
+        // contenuto: Flickable NON interattivo (lo scroll è solo programmatico).
+        // La finestra di lettura VIVE DENTRO i margini verticali (top/bottom
+        // margin): il testo non puo' fisicamente uscire dai bordi, quindi i
+        // margini alto/basso sono identici su OGNI pagina (non solo a inizio
+        // capitolo) e si aggiornano live con l'anteprima del dialog.
        Flickable {
            id: innerFlick
            anchors.fill: parent
+            anchors.topMargin: page.previewMarginT >= 0
+                               ? page.previewMarginT : appSettings.readerMarginV
+            anchors.bottomMargin: page.previewMarginT >= 0
+                                  ? page.previewMarginT : appSettings.readerMarginV
            contentHeight: readerContent.height
            interactive: false
            clip: true
@@ -154,15 +161,25 @@ Page {
            Item {
                id: readerContent
                width: parent.width
-                height: Math.max(parent.height,
-                                 textItem.height + 2 * (page.previewMarginT >= 0
-                                                        ? page.previewMarginT
-                                                        : appSettings.readerMarginV))
+                // Altezza = testo + "deficit" dell'ultima pagina: max-scroll
+                // diventa k*step esatto e l'ultima pagina e' allineata (niente
+                // testo ripetuto tra penultima e ultima). Calcolo INLINE nel
+                // binding: si rivaluta quando cambia la finestra (rotazione,
+                // anteprima margini) o l'altezza del testo (reflow).
+                height: {
+                    var step = Math.max(1, parent.height)
+                    var t = textItem.height
+                    var pad = 0
+                    if (t > 0) {
+                        var r = t % step
+                        pad = r === 0 ? 0 : step - r
+                    }
+                    return Math.max(parent.height, t + pad)
+                }

                Text {
                    id: textItem
                    x: page.previewMarginL >= 0 ? page.previewMarginL : appSettings.readerMargin
-                    y: page.previewMarginT >= 0 ? page.previewMarginT : appSettings.readerMarginV
                    width: parent.width - 2 * (page.previewMarginL >= 0
                                               ? page.previewMarginL
                                               : appSettings.readerMargin)
@@ -270,25 +287,20 @@ Page {
    function readScrollPosition() {
        if (!page.ready)
            return
-        // passo di pagina = viewport - 2*margine verticale: ogni pagina mostra
-        // il testo DENTRO i margini (niente righe a filo del bordo)
-        var step = Math.max(1, innerFlick.height - 2 * page.marginV())
+        // passo di pagina = altezza della finestra di lettura (i margini
+        // alto/basso sono FUORI dal Flickable: nessun testo li attraversa)
+        var step = Math.max(1, innerFlick.height)
        var max = Math.max(0, innerFlick.contentHeight - innerFlick.height)
        var y = Math.max(0, Math.min(max, innerFlick.contentY))
        var len = book.chapterTextLength(page.currentChapter)
-        page.chapterFraction = max > 0 ? y / max : 0
        // posizione per caratteri (stima dalla frazione: stabile col reflow)
-        page.charOffset = Math.round(page.chapterFraction * len)
+        page.charOffset = Math.round((max > 0 ? y / max : 0) * len)
        page.pageIndex = Math.floor(y / step) + 1
        page.pageCount = Math.max(1, Math.ceil(max / step) + 1)
+        updateCoverState()
        maybeSave(false)
    }

-    // margine verticale effettivo (anteprima dialog oppure impostazioni)
-    function marginV() {
-        return page.previewMarginT >= 0 ? page.previewMarginT : appSettings.readerMarginV
-    }
-
    function maybeSave(force) {
        if (bookId <= 0)
            return
@@ -308,7 +320,6 @@ Page {
            return
        if (index !== page.currentChapter) {
            maybeSave(true)
-            page.chapterFraction = 0
            page.charOffset = 0
            page.lastSavedAbs = -1
        }
@@ -335,13 +346,12 @@ Page {
        restoreTimer.start()
    }

-    // pagina successiva: una pagina di testo = viewport meno i margini verticali
-    // (così ogni pagina ha il testo dentro i margini, non a filo del bordo);
-    // a fine capitolo passa al successivo
+    // pagina successiva: una pagina di testo = altezza della finestra di
+    // lettura; a fine capitolo passa al successivo
    function nextPage() {
        if (!page.ready)
            return
-        var step = Math.max(1, innerFlick.height - 2 * page.marginV())
+        var step = Math.max(1, innerFlick.height)
        var max = Math.max(0, innerFlick.contentHeight - innerFlick.height)
        var y = innerFlick.contentY
        if (y >= max - 2) {
@@ -360,7 +370,7 @@ Page {
    function prevPage() {
        if (!page.ready)
            return
-        var step = Math.max(1, innerFlick.height - 2 * page.marginV())
+        var step = Math.max(1, innerFlick.height)
        var y = innerFlick.contentY
        if (y > 1) {
            innerFlick.contentY = Math.max(0, y - step)
@@ -368,7 +378,6 @@ Page {
        } else if (page.currentChapter > 0) {
            maybeSave(true)
            page.currentChapter -= 1
-            page.chapterFraction = 0
            page.charOffset = 0
            page.lastSavedAbs = -1
            // sentinella "fine capitolo": va oltre la lunghezza, così
@@ -383,6 +392,17 @@ Page {
        return Math.round(16 * page.fontScale * 100) / 100
    }

+    // aggiorna lo stato per la cover (titolo + capitolo + percentuale)
+    function updateCoverState() {
+        if (book.chapterCount <= 0)
+            return
+        var pct = Math.round(page.overallProgress() * 100)
+        readerState.setReading(book.title(),
+            (page.currentChapter + 1) + "/" + book.chapterCount
+            + " · " + book.chapterTitle(page.currentChapter)
+            + " · " + pct + "%")
+    }
+
    function showChapterList() {
        var dialog = pageStack.push(Qt.resolvedUrl("ChapterListDialog.qml"),
                                    { model: book.chapterList() })
@@ -424,11 +444,17 @@ Page {
        page.currentChapter = ch
        page.pendingRestore = true
        page.restoreFrac = off >= 0 ? offsetToFrac(off, ch) : 0.0
+        updateCoverState()
        restoreTimer.start()
    }

+    Component.onDestruction: readerState.clearReading()
+
    onStatusChanged: {
-        if (status === PageStatus.Inactive) {
+        if (status === PageStatus.Active) {
+            if (page.ready && book.chapterCount > 0)
+                updateCoverState()
+        } else if (status === PageStatus.Inactive) {
            maybeSave(true)
        }
    }
@@ -4,6 +4,8 @@ import Sailfish.Silica 1.0
 Page {
    id: page

+    allowedOrientations: Orientation.Portrait
+
    SilicaFlickable {
        anchors.fill: parent
        contentHeight: column.height
@@ -4,6 +4,8 @@ import Sailfish.Silica 1.0
 Page {
    id: page

+    allowedOrientations: Orientation.Portrait
+
    property string testResult: ""
    property bool testOk: false
    property bool testing: false
@@ -58,32 +60,6 @@ Page {
                onCheckedChanged: appSettings.ignoreSslErrors = checked
            }

-            SectionHeader { text: qsTr("Reader") }
-
-            Slider {
-                id: marginSlider
-                width: parent.width
-                label: qsTr("Side margins")
-                minimumValue: 0
-                maximumValue: 80
-                stepSize: 4
-                value: appSettings.readerMargin
-                valueText: value + " px"
-                onValueChanged: appSettings.readerMargin = value
-            }
-
-            Slider {
-                id: marginVSlider
-                width: parent.width
-                label: qsTr("Top and bottom margin")
-                minimumValue: 0
-                maximumValue: 80
-                stepSize: 4
-                value: appSettings.readerMarginV
-                valueText: value + " px"
-                onValueChanged: appSettings.readerMarginV = value
-            }
-
            SectionHeader { text: qsTr("Connection") }

            Button {
@@ -171,11 +147,18 @@ Page {
        if (base.indexOf("http://") !== 0 && base.indexOf("https://") !== 0)
            base = "https://" + base
        appSettings.serverUrl = base
+        appSettings.flush() // i campi di testo scrivono senza sync a ogni tasto
        page.testing = true
        page.testResult = qsTr("Testing…")
        apiClient.getFeed(base + "/opds")
    }

+    // uscita dalla pagina: scrivi su disco le modifiche in sospeso
+    onStatusChanged: {
+        if (status === PageStatus.Inactive)
+            appSettings.flush()
+    }
+
    Connections {
        target: apiClient
        onFeedReady: {
@@ -1,6 +1,6 @@
 Name:       harbour-calibreweb
 Summary:    Client OPDS per Calibre Web
-Version:    0.4.7
+Version:    0.4.11
 Release:    1
 Group:      Qt/Qt
 License:    MIT
@@ -24,6 +24,41 @@ reflow del testo, margini configurabili e posizione salvata
 in locale.

 %changelog
+* Fri Aug 21 2026 Carlo Baratto <carlo@carlobaratto.it> - 0.4.11-1
+- Rimossi gli slider margini dalle Impostazioni: i margini si regolano solo dal
+  lettore (Margini del testo nel PullDownMenu)
+- Margini alto/basso STRUTTURALI: la finestra di lettura vive dentro i margini
+  (top/bottom margin del Flickable): il testo non puo' uscire dai bordi e i
+  margini sono identici su ogni pagina (via le fasce colorate della 0.4.9/0.4.10)
+
+* Fri Aug 21 2026 Carlo Baratto <carlo@carlobaratto.it> - 0.4.10-1
+- Fix margini lettore (0.4.9): sfondo di lettura UNIFORME nel lettore (colore
+  pieno per tema, niente gradiente di sistema). Le fasce dei margini usano lo
+  stesso identico colore dello sfondo: margine visibile su ogni pagina, senza
+  bande stonate e senza effetto "fascia che copre il testo"
+
+* Fri Aug 21 2026 Carlo Baratto <carlo@carlobaratto.it> - 0.4.9-1
+- Margini verticali su OGNI schermata del lettore (non solo a inizio capitolo):
+  due fasce opache (colore di sfondo) sopra e sotto la viewport coprono il testo
+  a cavallo dei bordi; il padding inferiore del contenitore include il deficit
+  dell'ultima pagina, cosi' max-scroll e' un multiplo esatto del passo e
+  l'ultima pagina non ripete righe della penultima. Anteprima live dal dialog
+  margini inclusa
+
+* Fri Aug 21 2026 Carlo Baratto <carlo@carlobaratto.it> - 0.4.8-1
+- Best practice audit: orientazioni per pagina (finestra senza vincoli, il
+  lettore ora ruota davvero in landscape; pagine di lista in portrait;
+  rimossa la proprieta' privata _defaultPageOrientations)
+- Licenza MIT installata nel pacchetto (/usr/share/licenses/harbour-calibreweb)
+- EPUB: lunghezze testo LAZY per capitolo (l'apertura non parse piu' tutto il
+  libro: niente freeze sui libri grossi; cache al primo accesso)
+- Cover dinamica: con un libro aperto mostra titolo e posizione di lettura
+  (ReaderState esposto come context property, aggiornato dal ReaderPage)
+- Impostazioni: niente sync su disco a ogni tasto nei campi di testo
+  (flush() all'uscita della pagina e prima del test connessione)
+- Pulizia: rimossa proprieta' morta chapterFraction, commenti obsoleti
+  aggiornati (settings.h, .pro)
+
 * Fri Aug 21 2026 Carlo Baratto <carlo@carlobaratto.it> - 0.4.7-1
 - Fix margini lettore: il passo di pagina era la viewport intera, quindi il
  testo scorreva a filo dei bordi e i margini verticali si vedevano solo
@@ -199,6 +234,19 @@ desktop-file-validate %{buildroot}%{_datadir}/applications/%{name}.desktop || tr
 mkdir -p %{buildroot}%{_datadir}/%{name}/translations
 install -m 0644 %{_builddir}/%{name}_it.qm %{buildroot}%{_datadir}/%{name}/translations/

+# MIT license: installed in the package (Harbour best practice).
+mkdir -p %{buildroot}%{_datadir}/licenses/%{name}
+if [ -f %{_builddir}/../%{name}-%{version}/LICENSE ]; then
+    install -m 0644 %{_builddir}/../%{name}-%{version}/LICENSE %{buildroot}%{_datadir}/licenses/%{name}/
+elif [ -f %{_builddir}/%{name}-%{version}/LICENSE ]; then
+    install -m 0644 %{_builddir}/%{name}-%{version}/LICENSE %{buildroot}%{_datadir}/licenses/%{name}/
+elif [ -f %{_builddir}/LICENSE ]; then
+    install -m 0644 %{_builddir}/LICENSE %{buildroot}%{_datadir}/licenses/%{name}/
+else
+    echo "ERROR: LICENSE not found" >&2
+    exit 1
+fi
+
 %files
 %defattr(-,root,root,-)
 %{_bindir}/%{name}
@@ -209,3 +257,4 @@ install -m 0644 %{_builddir}/%{name}_it.qm %{buildroot}%{_datadir}/%{name}/trans
 %{_datadir}/icons/hicolor/172x172/apps/%{name}.png
 %{_datadir}/%{name}/qml
 %{_datadir}/%{name}/translations/%{name}*.qm
+%{_datadir}/licenses/%{name}/LICENSE
@@ -490,8 +490,14 @@ bool EpubBook::open(const QString &epubPath)
    if (!parseOpf())
        return false;
    const bool ok = !m_chapters.isEmpty();
-    if (ok)
-        computeTextLengths();
+    if (ok) {
+        // lunghezze testo LAZY: -1 = da calcolare al primo accesso. L'apertura
+        // resta veloce anche per libri grossi (niente parse di tutti i capitoli).
+        m_textLengths.clear();
+        m_textLengths.reserve(m_chapters.size());
+        for (int i = 0; i < m_chapters.size(); ++i)
+            m_textLengths.append(-1);
+    }
    emit changed();
    return ok;
 }
@@ -763,24 +769,26 @@ QString EpubBook::chapterRichText(int index) const
    return htmlToRichText(html);
 }

-void EpubBook::computeTextLengths()
+// calcola (e cache) la lunghezza del capitolo index se non ancora nota
+void EpubBook::ensureTextLength(int index) const
 {
-    m_textLengths.clear();
-    for (const EpubChapter &c : m_chapters) {
-        QFile f(c.path);
-        if (!f.open(QIODevice::ReadOnly)) {
-            m_textLengths.append(0);
-            continue;
-        }
-        const QString html = QString::fromUtf8(f.readAll());
-        m_textLengths.append(plainTextLength(html));
+    if (index < 0 || index >= m_textLengths.size() || m_textLengths.at(index) >= 0)
+        return;
+    const EpubChapter &c = m_chapters.at(index);
+    QFile f(c.path);
+    if (!f.open(QIODevice::ReadOnly)) {
+        m_textLengths[index] = 0;
+        return;
    }
+    const QString html = QString::fromUtf8(f.readAll());
+    m_textLengths[index] = plainTextLength(html);
 }

 int EpubBook::chapterTextLength(int index) const
 {
    if (index < 0 || index >= m_textLengths.size())
        return 0;
+    ensureTextLength(index);
    return m_textLengths.at(index);
 }

@@ -788,8 +796,10 @@ qint64 EpubBook::chapterStartChar(int index) const
 {
    qint64 start = 0;
    const int end = qMin(index, m_textLengths.size());
-    for (int i = 0; i < end; ++i)
+    for (int i = 0; i < end; ++i) {
+        ensureTextLength(i);
        start += m_textLengths.at(i);
+    }
    return start;
 }

@@ -66,15 +66,17 @@ private:
    void parseTocEpub3(const QString &navPath);
    void parseTocEpub2(const QString &ncxPath);
    QString absolutePath(const QString &relativeToOpfDir) const;
-    // lunghezze in caratteri del testo visibile, calcolate all'open
-    void computeTextLengths();
+    // calcola (e cache) la lunghezza in caratteri del capitolo al primo accesso
+    void ensureTextLength(int index) const;

    QString m_baseDir;
    QString m_opfDir;
    QString m_title;
    QString m_author;
    QList<EpubChapter> m_chapters;
-    QList<int> m_textLengths;
+    // lunghezze testo per capitolo, calcolate LAZY al primo accesso
+    // (-1 = non ancora calcolata); mutable per la cache da metodi const
+    mutable QList<int> m_textLengths;
 };

 #endif // EPUB_H
@@ -10,6 +10,7 @@
 #include "apiclient.h"
 #include "downloader.h"
 #include "epub.h"
+#include "readerstate.h"
 #include "settings.h"

 Q_DECL_EXPORT int main(int argc, char *argv[])
@@ -22,7 +23,7 @@ Q_DECL_EXPORT int main(int argc, char *argv[])
    app->setOrganizationName(QStringLiteral("harbour"));
    app->setApplicationName(QStringLiteral("calibreweb"));
    // versione mostrata nelle impostazioni (tenere allineata allo spec)
-    app->setApplicationVersion(QStringLiteral("0.4.7"));
+    app->setApplicationVersion(QStringLiteral("0.4.11"));

    Settings settings;

@@ -56,16 +57,19 @@ Q_DECL_EXPORT int main(int argc, char *argv[])

    QQuickView *view = SailfishApp::createView();

-    qDebug() << "harbour-calibreweb v0.4.7 build" << __DATE__ << __TIME__;
+    qDebug() << "harbour-calibreweb v0.4.11 build" << __DATE__ << __TIME__;

    qmlRegisterType<EpubBook>("harbour.calibreweb", 1, 0, "EpubBook");

    ApiClient apiClient(&settings);
    Downloader downloader(&settings);
+    // stato di lettura corrente (per la cover quando l'app va in background)
+    ReaderState readerState(&settings);

    view->rootContext()->setContextProperty(QStringLiteral("appSettings"), &settings);
    view->rootContext()->setContextProperty(QStringLiteral("apiClient"), &apiClient);
    view->rootContext()->setContextProperty(QStringLiteral("downloader"), &downloader);
+    view->rootContext()->setContextProperty(QStringLiteral("readerState"), &readerState);
    view->rootContext()->setContextProperty(QStringLiteral("appVersion"),
                                            QCoreApplication::applicationVersion());

@@ -0,0 +1,27 @@
+#include "readerstate.h"
+
+ReaderState::ReaderState(QObject *parent)
+    : QObject(parent)
+    , m_active(false)
+{
+}
+
+void ReaderState::setReading(const QString &title, const QString &subtitle)
+{
+    if (m_active && m_title == title && m_subtitle == subtitle)
+        return;
+    m_active = true;
+    m_title = title;
+    m_subtitle = subtitle;
+    emit changed();
+}
+
+void ReaderState::clearReading()
+{
+    if (!m_active)
+        return;
+    m_active = false;
+    m_title.clear();
+    m_subtitle.clear();
+    emit changed();
+}
@@ -0,0 +1,38 @@
+#ifndef READERSTATE_H
+#define READERSTATE_H
+
+#include <QObject>
+#include <QString>
+
+// Stato di lettura corrente, esposto come context property "readerState":
+// la cover mostra il libro aperto (titolo + posizione) invece dell'icona fissa.
+class ReaderState : public QObject
+{
+    Q_OBJECT
+
+    Q_PROPERTY(bool active READ active NOTIFY changed)
+    Q_PROPERTY(QString title READ title NOTIFY changed)
+    Q_PROPERTY(QString subtitle READ subtitle NOTIFY changed)
+
+public:
+    explicit ReaderState(QObject *parent = 0);
+
+    bool active() const { return m_active; }
+    QString title() const { return m_title; }
+    QString subtitle() const { return m_subtitle; }
+
+    // aggiorna lo stato quando il lettore mostra un libro (chiamata dal QML)
+    Q_INVOKABLE void setReading(const QString &title, const QString &subtitle);
+    // il libro è stato chiuso: la cover torna all'icona predefinita
+    Q_INVOKABLE void clearReading();
+
+signals:
+    void changed();
+
+private:
+    bool m_active;
+    QString m_title;
+    QString m_subtitle;
+};
+
+#endif // READERSTATE_H
@@ -115,9 +115,9 @@ void Settings::setServerUrl(const QString &value)
 {
    if (value == serverUrl())
        return;
+    // niente sync a ogni tasto (flash wear): il flush avviene all'uscita
+    // dalle impostazioni o prima delle richieste di rete (vedi flush())
    m_settings.setValue(QStringLiteral("serverUrl"), value);
-    m_settings.sync(); // scrivi subito su disco: l'app può crashare alla chiusura (WebView)
-    openPermissions();
    emit serverUrlChanged();
    emit baseUrlChanged();
 }
@@ -132,8 +132,6 @@ void Settings::setUsername(const QString &value)
    if (value == username())
        return;
    m_settings.setValue(QStringLiteral("username"), value);
-    m_settings.sync();
-    openPermissions();
    emit usernameChanged();
 }

@@ -147,9 +145,13 @@ void Settings::setPassword(const QString &value)
    if (value == password())
        return;
    m_settings.setValue(QStringLiteral("password"), value);
+    emit passwordChanged();
+}
+
+void Settings::flush()
+{
    m_settings.sync();
    openPermissions();
-    emit passwordChanged();
 }

 bool Settings::ignoreSslErrors() const
@@ -55,6 +55,10 @@ public:

    QString downloadDir() const;

+    // scrive subito su disco le modifiche in sospeso (i setter di testo non
+    // fanno sync a ogni tasto: chiamare questo all'uscita dalle impostazioni)
+    Q_INVOKABLE void flush();
+
    // Header "Authorization: Basic ..." pronto da usare, vuoto se utente non impostato
    QString authHeader() const;

@@ -80,17 +84,18 @@ signals:
    void baseUrlChanged();

 private:
-    // Path fisso condiviso: l'app deve leggere/scrivere SEMPRE lo stesso file,
-    // sia avviata dal launcher (utente nemo) sia da SSH di debug (defaultuser).
-    // Con QSettings("harbour","calibreweb") il file dipende dalla HOME
-    // dell'utente che avvia -> configurazione invisibile dall'altro utente.
+    // Path canonico per la sandbox SailJail (Sailfish 4.4+/5.x): con org/app
+    // impostati (harbour/calibreweb, vedi main.cpp e X-Sailjail nel desktop
+    // file), AppConfigLocation = ~/.config/harbour/calibreweb, l'unica zona
+    // config PERSISTENTE della sandbox (senza profilo la sandbox la nasconde).
    static QString configPath();
-    // Importa i valori dalla vecchia config per-utente (~/.config/harbour/...) se
-    // il file condiviso e' vuoto: copre la configurazione fatta prima del fix.
+    // Importa i valori dal vecchio file per-utente (~/.config/harbour/...) se
+    // il file canonico e' vuoto: copre la configurazione fatta prima del fix.
    void migrateIfNeeded();
-    // Apre i permessi della catena /home/nemo/.config/harbour e del file:
-    // il launcher gira come nemo (home 700), il debug SSH come defaultuser —
-    // entrambi devono poter leggere/scrivere lo stesso file di configurazione.
+    // Apre i permessi della catena di directory del file di configurazione:
+    // l'avvio dal launcher e quello da SSH (defaultuser) devono poter
+    // leggere/scrivere lo stesso file. Efficace quando l'app gira come
+    // owner della home (dal launcher).
    void openPermissions();

    QSettings m_settings;
@@ -1,29 +1,30 @@
-<?xml version='1.0' encoding='utf-8'?>
+<?xml version="1.0" encoding="utf-8"?>
+<!DOCTYPE TS>
 <TS version="2.1" language="it_IT">
 <context>
    <name>BookDetailPage</name>
    <message>
-        <location filename="../qml/pages/BookDetailPage.qml" line="36" />
+        <location filename="../qml/pages/BookDetailPage.qml" line="38"/>
        <source>Read (EPUB)</source>
        <translation>Leggi (EPUB)</translation>
    </message>
    <message>
-        <location filename="../qml/pages/BookDetailPage.qml" line="60" />
+        <location filename="../qml/pages/BookDetailPage.qml" line="62"/>
        <source>Formats</source>
        <translation>Formati</translation>
    </message>
    <message>
-        <location filename="../qml/pages/BookDetailPage.qml" line="112" />
+        <location filename="../qml/pages/BookDetailPage.qml" line="114"/>
        <source>Downloading…</source>
        <translation>Download in corso…</translation>
    </message>
    <message>
-        <location filename="../qml/pages/BookDetailPage.qml" line="261" />
+        <location filename="../qml/pages/BookDetailPage.qml" line="263"/>
        <source>Saved to </source>
        <translation>Salvato in </translation>
    </message>
    <message>
-        <location filename="../qml/pages/BookDetailPage.qml" line="266" />
+        <location filename="../qml/pages/BookDetailPage.qml" line="268"/>
        <source>Error: </source>
        <translation>Errore: </translation>
    </message>
@@ -31,12 +32,12 @@
 <context>
    <name>ChapterListDialog</name>
    <message>
-        <location filename="../qml/pages/ChapterListDialog.qml" line="12" />
+        <location filename="../qml/pages/ChapterListDialog.qml" line="12"/>
        <source>Table of contents</source>
        <translation>Indice capitoli</translation>
    </message>
    <message>
-        <location filename="../qml/pages/ChapterListDialog.qml" line="38" />
+        <location filename="../qml/pages/ChapterListDialog.qml" line="38"/>
        <source>No chapters</source>
        <translation>Nessun capitolo</translation>
    </message>
@@ -44,7 +45,7 @@
 <context>
    <name>CoverPage</name>
    <message>
-        <location filename="../qml/cover/CoverPage.qml" line="18" />
+        <location filename="../qml/cover/CoverPage.qml" line="22"/>
        <source>Calibre Web</source>
        <translation>Calibre Web</translation>
    </message>
@@ -52,18 +53,18 @@
 <context>
    <name>FeedPage</name>
    <message>
-        <location filename="../qml/pages/FeedPage.qml" line="91" />
+        <location filename="../qml/pages/FeedPage.qml" line="93"/>
        <source>Refresh</source>
        <translation>Aggiorna</translation>
    </message>
    <message>
-        <location filename="../qml/pages/FeedPage.qml" line="95" />
+        <location filename="../qml/pages/FeedPage.qml" line="97"/>
        <source>Settings</source>
        <translation>Impostazioni</translation>
    </message>
    <message>
-        <location filename="../qml/pages/FeedPage.qml" line="102" />
-        <location filename="../qml/pages/FeedPage.qml" line="171" />
+        <location filename="../qml/pages/FeedPage.qml" line="104"/>
+        <location filename="../qml/pages/FeedPage.qml" line="173"/>
        <source>No results</source>
        <translation>Nessun risultato</translation>
    </message>
@@ -71,37 +72,37 @@
 <context>
    <name>MainPage</name>
    <message>
-        <location filename="../qml/pages/MainPage.qml" line="13" />
+        <location filename="../qml/pages/MainPage.qml" line="15"/>
        <source>Calibre Web</source>
        <translation>Calibre Web</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MainPage.qml" line="43" />
+        <location filename="../qml/pages/MainPage.qml" line="45"/>
        <source>Search</source>
        <translation>Cerca</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MainPage.qml" line="47" />
+        <location filename="../qml/pages/MainPage.qml" line="49"/>
        <source>Settings</source>
        <translation>Impostazioni</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MainPage.qml" line="51" />
+        <location filename="../qml/pages/MainPage.qml" line="53"/>
        <source>Refresh</source>
        <translation>Aggiorna</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MainPage.qml" line="60" />
+        <location filename="../qml/pages/MainPage.qml" line="62"/>
        <source>Open the settings and enter your Calibre Web address</source>
-        <translation>Apri le impostazioni e inserisci l'indirizzo del tuo Calibre Web</translation>
+        <translation>Apri le impostazioni e inserisci l&apos;indirizzo del tuo Calibre Web</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MainPage.qml" line="77" />
+        <location filename="../qml/pages/MainPage.qml" line="79"/>
        <source>Server not configured</source>
        <translation>Server non configurato</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MainPage.qml" line="100" />
+        <location filename="../qml/pages/MainPage.qml" line="102"/>
        <source>No sections found in the OPDS feed</source>
        <translation>Nessuna sezione trovata nel feed OPDS</translation>
    </message>
@@ -109,27 +110,27 @@
 <context>
    <name>MarginDialog</name>
    <message>
-        <location filename="../qml/pages/MarginDialog.qml" line="25" />
+        <location filename="../qml/pages/MarginDialog.qml" line="25"/>
        <source>Text margins</source>
        <translation>Margini del testo</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MarginDialog.qml" line="26" />
+        <location filename="../qml/pages/MarginDialog.qml" line="26"/>
        <source>Apply</source>
        <translation>Applica</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MarginDialog.qml" line="32" />
+        <location filename="../qml/pages/MarginDialog.qml" line="32"/>
        <source>Side margins</source>
        <translation>Margini laterali</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MarginDialog.qml" line="47" />
+        <location filename="../qml/pages/MarginDialog.qml" line="47"/>
        <source>Top and bottom</source>
        <translation>Alto e basso</translation>
    </message>
    <message>
-        <location filename="../qml/pages/MarginDialog.qml" line="62" />
+        <location filename="../qml/pages/MarginDialog.qml" line="62"/>
        <source>Changes apply immediately. Confirm to save them for all books.</source>
        <translation>Le modifiche si applicano subito al testo. Conferma per salvarle per tutti i libri.</translation>
    </message>
@@ -137,37 +138,27 @@
 <context>
    <name>ReaderPage</name>
    <message>
-        <location filename="../qml/pages/ReaderPage.qml" line="35" />
+        <location filename="../qml/pages/ReaderPage.qml" line="58"/>
        <source>Table of contents</source>
        <translation>Indice capitoli</translation>
    </message>
    <message>
-        <location filename="../qml/pages/ReaderPage.qml" line="39" />
+        <location filename="../qml/pages/ReaderPage.qml" line="62"/>
        <source>Text margins</source>
        <translation>Margini del testo</translation>
    </message>
    <message>
-        <location filename="../qml/pages/ReaderPage.qml" line="53" />
+        <location filename="../qml/pages/ReaderPage.qml" line="77"/>
        <source>Settings</source>
        <translation>Impostazioni</translation>
    </message>
    <message>
-        <location filename="../qml/pages/ReaderPage.qml" line="57" />
+        <location filename="../qml/pages/ReaderPage.qml" line="81"/>
        <source>Close book</source>
        <translation>Chiudi libro</translation>
    </message>
    <message>
-        <location filename="../qml/pages/ReaderPage.qml" line="170" />
-        <source>The reader is not responding (WebView). The EPUB file may be damaged.</source>
-        <translation>Il lettore non risponde (WebView). Il file EPUB potrebbe essere danneggiato.</translation>
-    </message>
-    <message>
-        <location filename="../qml/pages/ReaderPage.qml" line="293" />
-        <source>Chapter unreadable</source>
-        <translation>Capitolo non leggibile</translation>
-    </message>
-    <message>
-        <location filename="../qml/pages/ReaderPage.qml" line="342" />
+        <location filename="../qml/pages/ReaderPage.qml" line="427"/>
        <source>Unable to open the EPUB file</source>
        <translation>Impossibile aprire il libro EPUB</translation>
    </message>
@@ -175,23 +166,23 @@
 <context>
    <name>SearchPage</name>
    <message>
-        <location filename="../qml/pages/SearchPage.qml" line="15" />
-        <location filename="../qml/pages/SearchPage.qml" line="27" />
+        <location filename="../qml/pages/SearchPage.qml" line="17"/>
+        <location filename="../qml/pages/SearchPage.qml" line="29"/>
        <source>Search</source>
        <translation>Cerca</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SearchPage.qml" line="20" />
+        <location filename="../qml/pages/SearchPage.qml" line="22"/>
        <source>Title or author…</source>
        <translation>Titolo o autore…</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SearchPage.qml" line="35" />
+        <location filename="../qml/pages/SearchPage.qml" line="37"/>
        <source>Search uses the server engine (OPDS search).</source>
        <translation>La ricerca usa il motore del server (OPDS search).</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SearchPage.qml" line="50" />
+        <location filename="../qml/pages/SearchPage.qml" line="52"/>
        <source>Results: </source>
        <translation>Risultati: </translation>
    </message>
@@ -199,100 +190,85 @@
 <context>
    <name>SettingsPage</name>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="20" />
+        <location filename="../qml/pages/SettingsPage.qml" line="22"/>
        <source>Settings</source>
        <translation>Impostazioni</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="22" />
+        <location filename="../qml/pages/SettingsPage.qml" line="24"/>
        <source>Server</source>
        <translation>Server</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="28" />
+        <location filename="../qml/pages/SettingsPage.qml" line="30"/>
        <source>Calibre Web address</source>
        <translation>Indirizzo Calibre Web</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="39" />
+        <location filename="../qml/pages/SettingsPage.qml" line="41"/>
        <source>User (empty = anonymous access)</source>
        <translation>Utente (vuoto = accesso anonimo)</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="48" />
+        <location filename="../qml/pages/SettingsPage.qml" line="50"/>
        <source>Password</source>
        <translation>Password</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="55" />
+        <location filename="../qml/pages/SettingsPage.qml" line="57"/>
        <source>Ignore SSL certificate errors</source>
        <translation>Ignora errori certificato SSL</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="56" />
+        <location filename="../qml/pages/SettingsPage.qml" line="58"/>
        <source>Enable if the server uses a self-signed certificate</source>
        <translation>Da attivare se il server usa un certificato autofirmato</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="61" />
-        <source>Reader</source>
-        <translation>Lettore</translation>
-    </message>
-    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="66" />
-        <source>Side margins</source>
-        <translation>Margini laterali</translation>
-    </message>
-    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="78" />
-        <source>Top and bottom margin</source>
-        <translation>Margine alto e basso</translation>
-    </message>
-    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="87" />
+        <location filename="../qml/pages/SettingsPage.qml" line="63"/>
        <source>Connection</source>
        <translation>Connessione</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="91" />
+        <location filename="../qml/pages/SettingsPage.qml" line="67"/>
        <source>Test connection</source>
        <translation>Prova connessione</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="115" />
-        <location filename="../qml/pages/SettingsPage.qml" line="120" />
+        <location filename="../qml/pages/SettingsPage.qml" line="91"/>
+        <location filename="../qml/pages/SettingsPage.qml" line="96"/>
        <source>Language</source>
-        <translation type="finished">Lingua</translation>
+        <translation>Lingua</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="123" />
+        <location filename="../qml/pages/SettingsPage.qml" line="99"/>
        <source>System (default)</source>
-        <translation type="finished">Sistema (predefinita)</translation>
+        <translation>Sistema (predefinita)</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="131" />
+        <location filename="../qml/pages/SettingsPage.qml" line="107"/>
        <source>Language will be applied after restarting the app</source>
-        <translation type="finished">La lingua verrà applicata al riavvio dell'app</translation>
+        <translation>La lingua verrà applicata al riavvio dell&apos;app</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="159" />
+        <location filename="../qml/pages/SettingsPage.qml" line="142"/>
        <source>Enter the server address</source>
-        <translation>Inserisci l'indirizzo del server</translation>
+        <translation>Inserisci l&apos;indirizzo del server</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="156" />
+        <location filename="../qml/pages/SettingsPage.qml" line="132"/>
        <source>Version</source>
-        <translation type="finished">Versione</translation>
+        <translation>Versione</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="168" />
+        <location filename="../qml/pages/SettingsPage.qml" line="152"/>
        <source>Testing…</source>
        <translation>Verifica in corso…</translation>
    </message>
    <message>
-        <location filename="../qml/pages/SettingsPage.qml" line="179" />
+        <location filename="../qml/pages/SettingsPage.qml" line="169"/>
        <source>Connection OK: %1 sections found</source>
        <translation>Connessione OK: %1 sezioni trovate</translation>
    </message>
 </context>
-</TS>
+</TS>
Author	SHA1	Message	Date
kaneda	6fcf05a289	0.4.11: slider margini rimossi dalle Impostazioni (resta il dialog nel lettore); margini alto/basso strutturali — finestra di lettura dentro i margini, identici su ogni pagina	2026-09-07 17:47:57 +02:00
kaneda	9500c353a3	Lettore 0.4.10: sfondo di lettura uniforme (colore pieno per tema) — le fasce margini usano lo stesso colore: margine su ogni pagina senza bande ne' testo coperto percepibile	2026-09-07 17:39:28 +02:00
kaneda	aef3cc3409	Lettore 0.4.9: margini verticali su ogni schermata — fasce opache fisse sopra/sotto la viewport (colore di sfondo per tema) + padding deficit dell'ultima pagina (max-scroll multiplo esatto del passo: ultima pagina allineata, niente righe ripetute)	2026-09-07 17:31:07 +02:00
kaneda	041b9b9567	Audit best practice 0.4.8: orientazioni per pagina (reader in landscape), licenza nel pacchetto, EPUB lazy lengths, cover dinamica (ReaderState), flush config senza sync per tasto, pulizia chapterFraction/commenti	2026-09-03 14:16:18 +02:00