<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - glib-demo: Simplify the list view for annotations" href="https://bugs.freedesktop.org/show_bug.cgi?id=70982#c6">Comment # 6</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - glib-demo: Simplify the list view for annotations" href="https://bugs.freedesktop.org/show_bug.cgi?id=70982">bug 70982</a> from <a class="email" href="mailto:gpoo@gnome.org" title="Germán Poo-Caamaño <gpoo@gnome.org>"> Germán Poo-Caamaño</a> <pre>(In reply to <a href="show_bug.cgi?id=70982#c5">comment #5</a>) > Comment on <a href="attachment.cgi?id=89392" name="attach_89392" title="glib-demo: Simplify annotations list and its properties">attachment 89392</a> <a href="attachment.cgi?id=89392&action=edit" title="glib-demo: Simplify annotations list and its properties">[details]</a> <a href='page.cgi?id=splinter.html&bug=70982&attachment=89392'>[review]</a> [review] > glib-demo: Simplify annotations list and its properties > > Review of <a href="attachment.cgi?id=89392" name="attach_89392" title="glib-demo: Simplify annotations list and its properties">attachment 89392</a> <a href="attachment.cgi?id=89392&action=edit" title="glib-demo: Simplify annotations list and its properties">[details]</a> <a href='page.cgi?id=splinter.html&bug=70982&attachment=89392'>[review]</a> [review]: > ----------------------------------------------------------------- > > I'm not sure about this, I find it a bit confusing. This is what I see with > my spanish locale: > > Coords: (152,0,556,0) (152,0,556,0) > > Maybe we could use something like: > > Coords: X1: aa,bb X2: aa,bb Y1: aa,bb Y2: aa,bb > > Which is what we currently have in the list but moved to a label in the > properties. I was trying to save space, so no need to resize the sidebar pane and leaving space to the rendering page. I had forgotten about the locale thing, though. Maybe we can use semicolon instead a comma, something like: (152,0;556,0) (152,0;556,0) or a dash: (152,0-556,0) (152,0-556,0) FWIW, I was also inclined to removed the decimals, but the PDF has decimals (for the annotations added with other software). The ones created with cairo, are integers. > The coords information is mainly useful when debugging, but now that we have > the view also in the annots demo, it would be really useful to paint a focus > ring around the currently selected annot. But that could be done as a follow > up patch. Indeed, the reason to save space and avoid the scrollbars in the side pane was precisely because it was becoming a pain to debug the coordinates when I was testing it :-) The focus ring would be good. IIRC, I had it done in an Evince patch I have.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>