Article title is redundant, takes up space

Bug #308098 reported by GreenReaper
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Wikipedia Dump Reader
Incomplete
Undecided
Unassigned

Bug Description

The article title (along with the article close button) is displayed in large text above the rendered article. This takes up perhaps 15% of the vertical space on an 800x480 screen. This display is redundant, as the fact that it is the selected item in the list identifies the current article.

I recommend that the title be removed and the article be expanded so that it fills the space that was previously occupied by the title. The close button could be moved to the top of the article list, or displayed as a small X button above the selected list item, or changed to an X and placed next to the searchbox checkbox control (which could maybe be shortened to "search").

The article should also extend down to the very bottom of the screen - right now it only extends down to the top of the horizontal scrollbar on the article list.

description: updated
Revision history for this message
GreenReaper (greenreaper) wrote :

Attached an example of the Dump Reader in 800x480 on a Nokia N800 showing this problem. Bear in mind that this is in full-screen mode - the article space is even more limited (down to 6 lines of text) if it is not.

Revision history for this message
Benjamin Thyreau (benji2) wrote :

Hi,
Thanks for that screenshot, it's quite interesting, as i wondered if the program could run nicely on an Nokia N800.

I commited a slightly different layout for the window, with minors changes, such as fully filling the bottom area, and moving the title out (adding it at the beginning of the article text)

To switch to this mode, please change the new "smallerLayout" variable to True, in the same spirit as indicated with the "Q. Can i change the text size ?" question of the README. Of course, you need to be able to write the python file (ie. not in system-wide install or through the ubuntu package). Hope it helps.

I'd be intereted for more feedback on that, in the case i need to include it the trick on the README.

Revision history for this message
GreenReaper (greenreaper) wrote :

Now, if I don't use that switch, I get "Wrong base class of toplevel widget". Did you need to update the main Form3 as well?

The new layout is better, but it seems to have mysteriously disabled fullscreen mode for some reason.

Revision history for this message
Benjamin Thyreau (benji2) wrote :

Hi
I'm unsure what can be the cause of that. I didn't make any change in the form3.ui file, so the code logic for the case where self.smallerLayout is False shouldn't have changed. I couldn't reproduce any errors here with a fresh bzr checkout.
I also couldn't explain the lack of fullscreen mode. I created the new layout file from changing the old layout, except using a slightly later qt4 version (4.3.2).

Changed in wikipediadumpreader:
status: New → Incomplete
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.