Fixed annoying of-by-one bug in the selection
This commit is contained in:
parent
2ae3f3bf7e
commit
651558d6c4
1 changed files with 3 additions and 3 deletions
|
@ -257,11 +257,11 @@ PlaylistList::mousePressEvent (QMouseEvent *event)
|
||||||
if (m_selected->count () > 0) {
|
if (m_selected->count () > 0) {
|
||||||
int o = m_selected->last ();
|
int o = m_selected->last ();
|
||||||
if (o < i) {
|
if (o < i) {
|
||||||
for (int y = o; y <= i; y++) {
|
for (int y = o+1; y <= i; y++) {
|
||||||
m_selected->append (y);
|
m_selected->append (y);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
for (int y = i; y <= o; y++) {
|
for (int y = i; y < o; y++) {
|
||||||
m_selected->append (y);
|
m_selected->append (y);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -444,7 +444,7 @@ PlaylistList::keyPressEvent (QKeyEvent *event)
|
||||||
{
|
{
|
||||||
/* Sort list and remove in reverse order */
|
/* Sort list and remove in reverse order */
|
||||||
qSort (*m_selected);
|
qSort (*m_selected);
|
||||||
for (int i = m_selected->count () - 1; i >= 0; i --) {
|
for (int i = (m_selected->count () - 1); i >= 0; i --) {
|
||||||
xmmsh->playlistRemove (m_selected->value (i));
|
xmmsh->playlistRemove (m_selected->value (i));
|
||||||
}
|
}
|
||||||
m_selected->clear ();
|
m_selected->clear ();
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue