GUI preview: fix issue with displaying some Tamil text

This commit is contained in:
Jean-Francois Dockes 2019-09-24 08:55:37 +02:00
parent 336dc0dc48
commit a88549452d

View File

@ -1,4 +1,4 @@
/* Copyright (C) 2014 J.F.Dockes /* Copyright (C) 2014-2019 J.F.Dockes
* This program is free software; you can redistribute it and/or modify * This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by * it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or * the Free Software Foundation; either version 2 of the License, or
@ -71,22 +71,28 @@ string PlainToRichQtPreview::PlainToRichQtPreview::header()
string PlainToRichQtPreview::startMatch(unsigned int grpidx) string PlainToRichQtPreview::startMatch(unsigned int grpidx)
{ {
LOGDEB2("startMatch, grpidx " << (grpidx) << "\n" ); LOGDEB2("startMatch, grpidx " << grpidx << "\n");
grpidx = m_hdata->index_term_groups[grpidx].grpsugidx; grpidx = m_hdata->index_term_groups[grpidx].grpsugidx;
LOGDEB2("startMatch, ugrpidx " << (grpidx) << "\n" ); LOGDEB2("startMatch, ugrpidx " << grpidx << "\n");
m_groupanchors[grpidx].push_back(++m_lastanchor); m_groupanchors[grpidx].push_back(++m_lastanchor);
m_groupcuranchors[grpidx] = 0; m_groupcuranchors[grpidx] = 0;
return string("<span style='"). // We used to create the region as:
append(qs2utf8s(prefs.qtermstyle)). // <span style="..."><a name="...">term</a></span>
append("'>"). // For some reason, this caused problems with the display of some
append("<a name=\""). // Tamil text (qt bug?). Just inserting a space character after
append(termAnchorName(m_lastanchor)). // the opening <a tag, before the text, clears the problem, reason
append("\">"); // unknown. We also inverted the <span and <a tags to avoid
// highlighting the spurious space.
string startmarker{
"<a name='" + termAnchorName(m_lastanchor) + "'> " +
"<span style='" + qs2utf8s(prefs.qtermstyle) + "'>"
};
return startmarker;
} }
string PlainToRichQtPreview::endMatch() string PlainToRichQtPreview::endMatch()
{ {
return string("</a></span>"); return "</span></a>";
} }
string PlainToRichQtPreview::termAnchorName(int i) const string PlainToRichQtPreview::termAnchorName(int i) const
@ -104,7 +110,7 @@ string PlainToRichQtPreview::startChunk()
int PlainToRichQtPreview::nextAnchorNum(int grpidx) int PlainToRichQtPreview::nextAnchorNum(int grpidx)
{ {
LOGDEB2("nextAnchorNum: group " << (grpidx) << "\n" ); LOGDEB2("nextAnchorNum: group " << grpidx << "\n");
map<unsigned int, unsigned int>::iterator curit = map<unsigned int, unsigned int>::iterator curit =
m_groupcuranchors.find(grpidx); m_groupcuranchors.find(grpidx);
map<unsigned int, vector<int> >::iterator vecit = map<unsigned int, vector<int> >::iterator vecit =
@ -121,7 +127,7 @@ int PlainToRichQtPreview::nextAnchorNum(int grpidx)
else else
m_groupcuranchors[grpidx]++; m_groupcuranchors[grpidx]++;
m_curanchor = vecit->second[m_groupcuranchors[grpidx]]; m_curanchor = vecit->second[m_groupcuranchors[grpidx]];
LOGDEB2("nextAnchorNum: curanchor now " << (m_curanchor) << "\n" ); LOGDEB2("nextAnchorNum: curanchor now " << m_curanchor << "\n");
} }
return m_curanchor; return m_curanchor;
} }
@ -182,4 +188,3 @@ void ToRichThread::run()
m_output.push_back(QString::fromUtf8(it->c_str(), it->length())); m_output.push_back(QString::fromUtf8(it->c_str(), it->length()));
} }
} }