<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#c7">Comment # 7</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:carlosgc@gnome.org" title="Carlos Garcia Campos <carlosgc@gnome.org>"> Carlos Garcia Campos</a> <pre>(In reply to <a href="show_bug.cgi?id=70982#c6">comment #6</a>) > (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] [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] [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. Yes, I prefer to keep two decimals, fwiw > > 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. Current evince already does that.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>