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

harbour-calibreweb.pro

@@ -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;
+# with lrelease and installs it into the app data dir
-# libsailfishapp loads harbour-calibreweb_<locale>.qm automatically.
+# (/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.
 

qml/cover/CoverPage.qml

@@ -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
+        }
+    }
 }

qml/harbour-calibreweb.qml

@@ -9,6 +9,7 @@ ApplicationWindow {
     initialPage: Component { MainPage {} }
     cover: Component { CoverPage {} }
 
-    allowedOrientations: Orientation.Portrait
+    // Niente vincoli a livello window: ogni pagina dichiara le proprie
-    _defaultPageOrientations: Orientation.Portrait
+    // orientazioni (il lettore ruota in landscape, le pagine di lista
+    // restano in portrait). _defaultPageOrientations è API privata.
 }

qml/pages/BookDetailPage.qml

@@ -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

qml/pages/FeedPage.qml

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

qml/pages/MainPage.qml

@@ -4,6 +4,8 @@ import Sailfish.Silica 1.0
 Page {
     id: page
 
+    allowedOrientations: Orientation.Portrait
+
     property bool loading: false
     property string errorMessage: ""
 

qml/pages/ReaderPage.qml

@@ -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,
+                // Altezza = testo + "deficit" dell'ultima pagina: max-scroll
-                                 textItem.height + 2 * (page.previewMarginT >= 0
+                // diventa k*step esatto e l'ultima pagina e' allineata (niente
-                                                        ? page.previewMarginT
+                // testo ripetuto tra penultima e ultima). Calcolo INLINE nel
-                                                        : appSettings.readerMarginV))
+                // 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
+        // passo di pagina = altezza della finestra di lettura (i margini
-        // il testo DENTRO i margini (niente righe a filo del bordo)
+        // alto/basso sono FUORI dal Flickable: nessun testo li attraversa)
-        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 = 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
+    // pagina successiva: una pagina di testo = altezza della finestra di
-    // (così ogni pagina ha il testo dentro i margini, non a filo del bordo);
+    // lettura; a fine capitolo passa al successivo
-    // 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)
         }
     }

qml/pages/SearchPage.qml

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

qml/pages/SettingsPage.qml

@@ -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: {

rpm/harbour-calibreweb.spec

@@ -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

src/epub.cpp

@@ -490,8 +490,14 @@ bool EpubBook::open(const QString &epubPath)
     if (!parseOpf())
         return false;
     const bool ok = !m_chapters.isEmpty();
-    if (ok)
+    if (ok) {
-        computeTextLengths();
+        // 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();
+    if (index < 0 || index >= m_textLengths.size() || m_textLengths.at(index) >= 0)
-    for (const EpubChapter &c : m_chapters) {
+        return;
+    const EpubChapter &c = m_chapters.at(index);
     QFile f(c.path);
     if (!f.open(QIODevice::ReadOnly)) {
-            m_textLengths.append(0);
+        m_textLengths[index] = 0;
-            continue;
+        return;
     }
     const QString html = QString::fromUtf8(f.readAll());
-        m_textLengths.append(plainTextLength(html));
+    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;
 }
 

src/epub.h

@@ -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
+    // calcola (e cache) la lunghezza in caratteri del capitolo al primo accesso
-    void computeTextLengths();
+    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

src/main.cpp

@@ -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());
 

src/readerstate.cpp

@@ -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();
+}

src/readerstate.h

@@ -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

src/settings.cpp

@@ -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

src/settings.h

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

translations/harbour-calibreweb_it.ts

@@ -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>
@@ -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="104"/>
-        <location filename="../qml/pages/FeedPage.qml" line="171" />
+        <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>
@@ -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" />
+        <location filename="../qml/pages/ReaderPage.qml" line="427"/>
-        <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" />
         <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="17"/>
-        <location filename="../qml/pages/SearchPage.qml" line="27" />
+        <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,98 +190,83 @@
 <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" />
+        <location filename="../qml/pages/SettingsPage.qml" line="63"/>
-        <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" />
         <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="91"/>
-        <location filename="../qml/pages/SettingsPage.qml" line="120" />
+        <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>