Hi, I just run tested the patch, and it looks good. Please see the comments below. I also commented on some parts which are not part of the new GUI, but just the user's perspective; maybe I should resend it in a separate email.
1) Main menu -> Help -> View log does nothing. If it's not implemented yet, I would hide it to avoid confusion.
2) Help button on the main window shows About dialog. We have no manual, no help texts. I would hide the button for the release version to avoid confusion.
3) "Copy to Clipboard" button when used on kerneloops: Traceback (most recent call last): File "/usr/share/abrt/CCMainWindow.py", line 322, in on_b_copy_clicked ("Comment:", dump.comment), AttributeError: Dump instance has no attribute 'comment'
4) "Copy to Clipboard" is enabled on kerneloops, but it is disabled on other regular crash (seems it's disabled on unreported crashes). I see no difference in the "Details" panel which would justify the disablement. Please enable it for all crashes.
5) When kerneloops is selected, there are some problems in the Details panel: a) The heading is "Kerne Crash" (missing letter) b) Under the heading there is a line with word "kernel", but when other packages are selected, the line contains package name with version number and release number. So for the kerneloops it should probably also show the version and release numbers of the kernel. c) the panel shows "Command: not_applicable". When the command does not make sense for kerneloops, it should be hidden. Or, it can say "not applicable" without the underscore. d) The "Reason:" part shows this text: "---------------[cut here]-------------------" It is very confusing, as this is not a "reason". I do not know if the Reason field makes sense for kerneloopses, but it clearly does not make sense in some cases.
6) In Details panel uppercase/lowercase is not consistent: compare "Latest _C_rash" and "Crash _c_ount"
7) Main menu -> Help -> About dialog shows two lines: (C) 2009 Red Hat, Inc. (C) 2010 Red Hat, Inc. The copyright lines should not be duplicated for every year. I would merge them into one line: (C) 2009, 2010 Red Hat, Inc.
8) Main menu -> Edit -> Plugins dialog: a) I would rename "Not loaded plugins" to "Unloaded plugins" b) A header "Name" on the top is misleading, as the list does not contain names. It should be removed. c) Plugin details are missing for unloaded plugins. From user's perspective there is no reason why those should be missing. d) Plugin details should not include "author" line. The line suggests user to send email to the provided email address and ask for help. The "Web Site" field is more relevant: if user wants to know more about some particular plugin, he visits the website and looks for help there. e) Kerneloops plugin shows website "https://people.redhat.com/aarapov". ABRT team provides support for the kerneloops plugin, so the link should probably direct user to ABRT wiki f) Version numbers displayed for various plugins are wrong. We do not update them even when the plugin is completely changed. So for our "core" plugins, I would synchronize the plugin version with ABRT version. g) Bugzilla plugin -> Configure contains "No SSL verify" checkbox. It should be "SSL verify" to avoid mandatory exercises in basic logic. h) Bugzilla plugin -> Configure contains "Login:" field. Several users were already confused by it and wrote wrong value into it, we should change it to "Login email:"
9) Main menu -> Edit -> Preferences is still visible even when it's not implemented. It already confused many users (they reported it as a bug). We should hide it at least for official RPMs.
10) When abrt-gui is started, the focus in on the "Application" column heading. The focus should be on the crash list, so user can instantly use arrow keys to select a crash.
11) When you press Tab key to move focus from one control to another, it jumps to some hidden control, but not to the Details panel. When the focus is on the crash list and user pressed Tab, it should jump to the Details panel (which is scrollable when the window is too small), and form the Details panel it should jum to the Delete button.
12) The "Help" and "Close" buttons on the main window should be moved upwards to retain the padding between the controls and the edge of the window. The buttons are too close to the status bar. Also the "Report" button is placed too close to the right border of the window. It should be aligned with other controls.
Best regards, Karel
On 03/25/2010 04:08 PM, Jiri Moskovcak wrote:
Hi, this is a patch to change the main window according to desing from Mairin Duffy. The screenshots can be found here:
http://jmoskovc.fedorapeople.org/icons_as_status.png
in case someone just want to see it.
Please review, Jirka
Crash-catcher mailing list Crash-catcher@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/crash-catcher