[Bug 244061] New: New UI::Glyph - Ballot
https://bugzilla.novell.com/show_bug.cgi?id=244061 Summary: New UI::Glyph - Ballot Product: openSUSE 10.3 Version: unspecified Platform: All OS/Version: Other Status: NEW Severity: Enhancement Priority: P5 - None Component: YaST2 AssignedTo: sh@novell.com ReportedBy: kmachalkova@novell.com QAContact: jsrain@novell.com CC: mvidner@novell.com I'd like to add a new UI::Glyph as a counterpart to CheckMark (indicating that some step has succesfully passed). It would be a little cross mark indicating failure (U+2718), which is useful e.g. in Internet test to give user feedback that certain step has failed (related bug #121705). See attached screenshot. Now the problem arises with ncurses (that use ASCII equivalents of these glyphs from y2-core). We use [x] as CheckMark, which is confusing, because it looks a bit like CheckBox (related bug #166237) Therefore, I've proposed [OK] string for CheckMark and [x] for Ballot However, I understand the limitations that proposed unicode character might not be in some fonts used by Qt and in that case, feel free to close as invalid - it's better to have no character at all than an empty square. I'll attach screenshots and patches for y2-qt and y2-core shortly -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 ------- Comment #1 from kmachalkova@novell.com 2007-02-09 07:09 MST ------- Created an attachment (id=118308) --> (https://bugzilla.novell.com/attachment.cgi?id=118308&action=view) qt screenshot -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 ------- Comment #2 from kmachalkova@novell.com 2007-02-09 07:10 MST ------- Created an attachment (id=118309) --> (https://bugzilla.novell.com/attachment.cgi?id=118309&action=view) ncurses screenshot -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 ------- Comment #3 from kmachalkova@novell.com 2007-02-09 07:16 MST ------- Created an attachment (id=118311) --> (https://bugzilla.novell.com/attachment.cgi?id=118311&action=view) Patch for libyui . and that reminds me - to settle bug #166237 we need something different indicating failure (not [x] as in my proposal - this still looks like checkbox) :-( -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 ------- Comment #4 from kmachalkova@novell.com 2007-02-09 07:17 MST ------- Created an attachment (id=118312) --> (https://bugzilla.novell.com/attachment.cgi?id=118312&action=view) patch fro y2-qt -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 locilka@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #118312|text/x-patch |text/plain mime type| | Attachment #118312|0 |1 is patch| | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 locilka@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #118311|text/x-patch |text/plain mime type| | Attachment #118311|0 |1 is patch| | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 sh@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mfabian@novell.com ------- Comment #5 from sh@novell.com 2007-02-09 07:25 MST ------- Looking back on the past couple of years of using those glyphs, frankly I am all against extending that concept any further. To the contrary, I'd rather drop them today. Please look at how many bugs those glyphs induced. With every release we had another couple of "glyphs missing" bugs, sometimes even a couple dozen. Mike Fabian can tell a lot of painful stories about that - he usually gets to fix those font issues. Glyphs sounded like a good idea at the time. But it did not pay off. It backfired all the time. They are a lot more trouble than they are worth. We should do other things to improve the looks. In Qt, we can use pixmaps and icons; the concepts are there. IIRC the `Image() widget even has a fallback text to be used for NCurses. This is the way to go. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 sh@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mschmidkunz@novell.com ------- Comment #6 from sh@novell.com 2007-02-09 07:28 MST ------- As for this particular glyph, I think the NCurses representation (from comment #2) is very misleading and counterproductive to usability: [x] Downloading the latest release notes... This looks exactly like an NCurses check box. But this it is not: It is a passive screen element. Users should not be confused about what is an interactive widget and what are static output-only elements. Martin? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 ------- Comment #7 from kmachalkova@novell.com 2007-02-09 07:59 MST ------- Anyway, I'd like to somehow give user the feedback that this particular step of the test has failed. Now exactly the same indicator is used both for success and for failure. That's why I originally proposed a new glyph. We use: dash - test not started yet arrow - test in progress checkmark - test passed or failed If the glyph is not a way to go (due to font problems), what about using some single ASCII character indicating failure ? ('!' character, for example) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 ------- Comment #8 from mfabian@novell.com 2007-02-09 08:10 MST ------- Availability of U+2718 (HEAVY BALLOT X) in the fonts in instsys: Gnu Unifont (Bitmap): yes Efont Unicode (Bitmap): no DejaVu Sans (Scalable): yes As you can see in bug #199997, Efont is sometimes used by Qt3 instead of Gnu Unifont for reasons unknown. Therefore, to make this glyph work reliably in YaST2, I would need to draw one and add it to Efont Unicode. With Qt4 missing glyph problems should be finally solved because Qt4 uses single glyph fallback, i.e. it is enough if there as one font available which has the glyph, Qt4 will then always find it. Qt3 may decide to use a font for a certain Unicode range and will *not* fall back to other fonts for glyphs missing in that font in that range. Therefore, all fonts which might be used by Qt3 for a certain range *must* have *all* glyphs which might become necessary. In case of the special symbols on the Unicode page starting with U+2700 Qt3 currently may use any of the fonts mentioned above depending on screen resolution and the air pressure in the office or the shoe size of the operator. Therefore, we have to make sure that all glyphs we use from that page are available in all 3 of these fonts. By the way, '☹' would be nicer for failure, wouldn't it? And this one is already available in all 3 fonts, all of them are rather complete on the U+2600 page. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=244061 mschmidkunz@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED ------- Comment #9 from mschmidkunz@novell.com 2007-02-13 06:10 MST ------- The user should be given a clear chance to see the destinction between success and failure. Therefore we definitly need two different signs for test passed or failed. In direct discussion with Katarina Machalkova we decided that the sign used in ncurses will be: dash - test not started yet arrow - test in progress x - test passed ! - test failed We choose the x because the user knows this sign as a sign for checked items in ncurses. By leaving off the bracktes it is now unobstrusive to a check box in ncurses. The ! is commonly used as a sign for warning. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
participants (1)
-
bugzilla_noreply@novell.com