Bug 147233 - no horizontal scroll bar in table of contents sidebar
Summary: no horizontal scroll bar in table of contents sidebar
Status: RESOLVED FIXED
Alias: None
Product: kpdf
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Albert Astals Cid
URL:
Keywords:
: 149239 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-06-26 11:19 UTC by Leo Savernik
Modified: 2009-08-22 03:42 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
current accessibility issue (4.80 KB, image/png)
2007-06-26 11:22 UTC, Leo Savernik
Details
patch (1.43 KB, patch)
2007-06-26 17:38 UTC, Leo Savernik
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Leo Savernik 2007-06-26 11:19:03 UTC
Version:            (using KDE KDE 3.5.7)
Installed from:    Compiled From Sources

PDFs with deeply nested tables of contents are not fully accessible in the contents sidebar as it provides no horizontal scrollbar on small window widths.

This leads to a chapter's sections being cut off instead of horizontally scrollable at a 120px sidebar width.

Steps to reproduce.
1. Load a document with an elaborate table of contents. I used http://java.sun.com/docs/books/jls/download/langspec-3.0.pdf
2. Reduce sidebar width to ~120px

Expected results:
A horizontal scrollbar appears, allowing me to center on the subsection list of a chapter

Actual results:
No horizontal scrollbar appears, subsection titles are cut off.
Comment 1 Leo Savernik 2007-06-26 11:22:21 UTC
Created attachment 20967 [details]
current accessibility issue

Table of contents exhibiting the issue. Note that entries are cut off insteasd
of providing a horizontal scrollbar.
Comment 2 Leo Savernik 2007-06-26 17:38:34 UTC
Created attachment 20973 [details]
patch

Patch fixing the bug.

Please review and commit before 3.5.8 or let me commit it.
Comment 3 Albert Astals Cid 2007-06-26 22:41:48 UTC
Hi Leo, we already provide a Tooltip for TOC items that don't fit in the TOC width, but if you think your patch is better please go ahead.
Comment 4 Pino Toscano 2007-06-26 22:47:03 UTC
What about enabling directly the horizontal scrollbar of the TOC, instead of using a dummy widget?
Comment 5 Leo Savernik 2007-06-27 10:38:25 UTC
@ comment 4: Been there, done that. The scrollbar will stretch beyond the visible viewport (seems to be an issue with QToolBox).
Comment 6 Leo Savernik 2007-06-27 10:40:54 UTC
SVN commit 680874 by savernik:

Provide horizontal scrollbar in table of contents sidebar instead of
cropping the entries. This improves usability esp. for deeply nested TOCs.

BUG: 147233


 M  +6 -2      part.cpp  
 M  +2 -1      ui/toc.cpp  


--- branches/KDE/3.5/kdegraphics/kpdf/part.cpp #680873:680874
@@ -186,9 +186,13 @@
 
 	int index;
 	// [left toolbox: Table of Contents] | []
-	m_tocFrame = new TOC( m_toolBox, m_document );
+	// dummy wrapper with layout to enable horizontal scroll bars (bug: 147233)
+	QWidget *tocWrapper = new QWidget(m_toolBox);
+	QVBoxLayout *tocWrapperLayout = new QVBoxLayout(tocWrapper);
+	m_tocFrame = new TOC( tocWrapper, m_document );
+	tocWrapperLayout->add(m_tocFrame);
 	connect(m_tocFrame, SIGNAL(hasTOC(bool)), this, SLOT(enableTOC(bool)));
-	index = m_toolBox->addItem( m_tocFrame, QIconSet(SmallIcon("text_left")), i18n("Contents") );
+	index = m_toolBox->addItem( tocWrapper, QIconSet(SmallIcon("text_left")), i18n("Contents") );
 	m_toolBox->setItemToolTip(index, i18n("Contents"));
 	enableTOC( false );
 
--- branches/KDE/3.5/kdegraphics/kpdf/ui/toc.cpp #680873:680874
@@ -64,7 +64,8 @@
 #endif
     setSorting(-1);
     setRootIsDecorated(true);
-    setResizeMode(AllColumns);
+    // the next line causes bug:147233
+//    setResizeMode(AllColumns);
     setAllColumnsShowFocus(true);
     connect(this, SIGNAL(clicked(QListViewItem *)), this, SLOT(slotExecuted(QListViewItem *)));
     connect(this, SIGNAL(returnPressed(QListViewItem *)), this, SLOT(slotExecuted(QListViewItem *)));
Comment 7 Albert Astals Cid 2007-08-26 20:06:25 UTC
*** Bug 149239 has been marked as a duplicate of this bug. ***
Comment 8 LandoSr 2009-08-22 03:42:29 UTC
(KPDF: 0.5.10 (using KDE 3.5.10 "release 21.11" openSUSE)
(Okular: Version 0.7.3 Using KDE 4.1.3 (KDE 4.1.3) "release 4.10.4")

Hi, it has been a while since this bug. I find the limited sizing of the left pane _with scrollbar_ very irritating (also in Okular). Hovering over the TOC entry does not show the whole entry in my openSUSE 11.1 64 Bit System, on which Okular does show the hover. 

Why can the two panes not be varied without limits for either the TOC or the main displayed PDF?