QPlainTextEdit et QTextBlock::setVisibility(bool b))

Version imprimable

14/04/2011, 20h14
Agoudard

QPlainTextEdit et QTextBlock::setVisibility(bool b))

Bonsoir,

Voilà j'ai un petit problème avec un QPlainTextEdit dont le QTextDocument contient des QTextBlock cachés.
J'ai plusieurs lignes successives cachées grâce à QTextBlock::setVisibility(bool b).
Il arrive que le scroll vertical ne fonctionne pas correctement, on dirait que QAbstractScrollArea ne tient pas compte des lignes cachées.

J'ai trouvé une ou deux références à ce problème sur des forums malheureusement sans solutions.

Quelqu'un aurait une idée pour éviter ce problème ?

Merci
14/04/2011, 23h12
Amnell

Bonsoir,

Après une petite recherche sur Google, je suis tombé sur le rapport de bug http://bugreports.qt.nokia.com/browse/QTBUG-9573 qui traite le problème d'un crash avec QTextBlock::setVisible(false), mais ce n'est apparemment pas le problème.

À première vue,

Citation:

Il arrive que le scroll vertical ne fonctionne pas correctement, on dirait que QAbstractScrollArea ne tient pas compte des lignes cachées.

indique qu'il s'agirait probablement d'un problème de peinture. Avez-vous tenté d'utiliser la fonction update() après avoir affiché votre QTextBlock ?

Bonne continuation,
Amnell.
15/04/2011, 00h58
Agoudard

Oui j'utilise update de ma sous classe de QPlainTextEdit et du viewport.
Et oui, il s'agit d'un problème de peinture puisque j'observe une copie de la première ligne après les blocs cachés sur la ligne suivante si je réalise un scroll avant arrière.

J'arrive à reproduire le bug mais c'est difficile à décrire.
Je peux éventuellement mettre le code mais c'est un peu long...

Qt Creator semble avoir recourt à la fonction setVisible() de QTextBlock. Voici un commit où ils le problème semble être traité :

http://qt.gitorious.org/+qtuitest/qt...8023963c2c9920

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
void BaseTextEditor::drawCollapsedBlockPopup(QPainter &painter,
                                             const QTextBlock &block,
                                             QPointF offset,
                                             const QRect &clip)
{
    int margin = block.document()->documentMargin();
    qreal maxWidth = 0;
    qreal blockHeight = 0;
    QTextBlock b = block;
 
    while (!b.isVisible()) {
        b.setVisible(true); // make sure block bounding rect works
        QRectF r = blockBoundingRect(b).translated(offset);
 
        QTextLayout *layout = b.layout();
        for (int i = layout->lineCount()-1; i >= 0; --i)
            maxWidth = qMax(maxWidth, layout->lineAt(i).naturalTextWidth() + 2*margin);
 
        blockHeight += r.height();
 
        b.setVisible(false); // restore previous state
        b.setLineCount(0); // restore 0 line count for invisible block
        b = b.next();
    }
 
    painter.save();
    painter.setRenderHint(QPainter::Antialiasing, true);
    painter.translate(.5, .5);
    QBrush brush = palette().base();
    if (d->m_ifdefedOutFormat.hasProperty(QTextFormat::BackgroundBrush))
        brush = d->m_ifdefedOutFormat.background();
    painter.setBrush(brush);
    painter.drawRoundedRect(QRectF(offset.x(),
        offset.y(),
        maxWidth, blockHeight).adjusted(0, 0, 0, 0), 3, 3);
    painter.restore();
 
    QTextBlock end = b;
    b = block;
    while (b != end) {
        b.setVisible(true); // make sure block bounding rect works
        QRectF r = blockBoundingRect(b).translated(offset);
        QTextLayout *layout = b.layout();
        QVector<QTextLayout::FormatRange> selections;
        layout->draw(&painter, offset, selections, clip);
 
        b.setVisible(false); // restore previous state
        offset.ry() += r.height();
        b = b.next();
    }
}

Le plus intéressant dans ce code, c'est sans doute l'utilisation de QTextLayout qui semble être une piste intéressante.

15/04/2011, 15h20
Agoudard

Euh, je pense qu'il s'agit du popup qui affiche le texte replié.
Pas de rapport avec le scrolling ni avec la méthode utilisée pour réaliser le repli.

Mais merci quand même...

En effet, j'étais complètement à côté de la plaque. J'ai réussi à reproduire le problème, et également à le corriger avec l'appel de repaint() sur le viewport() :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
#include <QtGui>
 
class MyClass : public QWidget
{
    Q_OBJECT
public:
    MyClass() {
        _textEdit = new QPlainTextEdit;
        QPushButton *_button = new QPushButton("Hide/Show first block");
        setLayout(new QVBoxLayout);
        layout()->addWidget(_button);
        layout()->addWidget(_textEdit);
        connect(_button, SIGNAL(clicked()), this, SLOT(hideOrShow()));
    }
private:
    QPlainTextEdit *_textEdit;
private slots:
    void hideOrShow() {
        QTextBlock _hiddenBlock = _textEdit->document()->firstBlock();
        if (!_hiddenBlock.isValid()) {
            QMessageBox::warning(this, "Error", "Invalid text block. Passed.");
            return;
        }
        _hiddenBlock.setVisible(!_hiddenBlock.isVisible());
        _textEdit->viewport()->repaint();
    }
 
};
 
#include "main.moc"
 
int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
 
    QMainWindow w;
    w.setCentralWidget(new MyClass);
    w.show();
 
    return a.exec();
}

15/04/2011, 16h09
Agoudard

Héhé c'est plus compliqué que ça, j'ai aussi un update sur le viewport sinon la maj de visibilité des blocs n'est pas pris en compte immédiatement.

Le mieux c'est peut être que je t'envois le code ? (Si tu as le temps bien sûr)
15/04/2011, 16h20
Amnell

Allons-y. Je vous laisse héberger cela ou poster à la suite, comme vous préférez.
15/04/2011, 16h33
Agoudard

http://www.filedropper.com/webed

C'est un zip, compile avec Qt 4.7 (QMdiArea) mais la partie qui pose problème c'est ce qu'il y a dans le dossier CodeEdit.

Le constructeur de CCodeControl ajoute pour tester des contrôles de repliage de code artificiels qui s'organisent correctement que lorsque la ligne existe. Il faut donc écrire au moins une 30 aine de lignes.

Pour reproduire le bug il faut plier le bloc 2 qui à la particularité d'être imbriqué dans le bloc 1. Puis il faut scroller verticalement jusqu'après ce bloc (il doit disparaître du viewport d'une ligne) puis rescroller vers le bas d'un cran et ainsi desuite.

Désolé pour la clarté de commentaire et merci de ton aide.

EDIT : Erreur il s'agit de plier le PREMIER bloc et de descendre jusqu'à ce qu'il ne défile pas correctement sur le coté gauche et à partir de là réaliser le va et vient avec le scroll.
15/04/2011, 17h18
Amnell

1 pièce(s) jointe(s)
On voit bien le problème, mais ce n'est pas le seul : les carrés de dépliement et de repliement sur le côté apparaissent lors d'un scroll (et ce n'est pas un problème de peinture).

http://www.developpez.net/forums/att...1&d=1302878935

Apparemment, il y a une superposition de boutons qui sont peints à la première ligne visible (une erreur quelque part, sans doute).

Ici, il y a eu un problème de peinture que je n'ai pas pu saisir avec la capture d'écran, puisque la perte de focus de la fenêtre à forcé le repaint. Avant cette perte de focus, on voyait ceci :
Code:

1 2 3 4 5 6 } painter.drawLine(width() / 2, 2, width() / 2, height() - 3); painter.drawLine(width() / 2, 2, width() / 2, height() - 3); } }
Avec la deuxième occurrence de la ligne étant une fausse ligne due au problème de peinture.

Pour le problème de peinture, j'arrive à le virer en faisant ceci :

Code:

connect(verticalScrollBar(), SIGNAL(valueChanged(int)), viewport(), SLOT(repaint()));

Dans le fichier CCodeEdit.cpp, dans les connexions.

Par contre, la scrollbar ne semble pas s'adapter à un changement de contenu de la zone quand un bloc est replié. Dans ce cas, elle "pédale dans la semoule" pendant qu'elle ne scrolle rien du texte, tentant de scroller le texte caché, jusqu'à arriver à scroller. Et là, un repaint ne suffit pas.

Espérant que cela vous aidera,
Amnell.
15/04/2011, 17h26
Agoudard

Merci beaucoup, j'y jette un coup d'oeil dès que possible.

EDIT : Effectivement la connexion résout le problème mais le scroll patine dans la semoule sur les blocs non visibles ....
15/04/2011, 17h38
Amnell

Pour cette histoire de patinage, il faudrait voir chez QPlainTextEdit & parents ainsi que chez QScrollBar & parents s'il n'y a pas un moyen de re-génération, puisque ce n'est évidemment pas un problème de peinture.
16/04/2011, 13h42
Agoudard

J'ai trouvé la solution :
Il faut utiliser QTextDocument::markContentsDirty(int position, int length) sur les blocs dont la visibilité à été modifiée.
Il n'est plus besoin alors de connecter le Scroll vertical au repaint().