<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - Add implementation of Square and Circle annotations"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=70983#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - Add implementation of Square and Circle annotations"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=70983">bug 70983</a>
              from <span class="vcard"><a class="email" href="mailto:carlosgc@gnome.org" title="Carlos Garcia Campos <carlosgc@gnome.org>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=70983#c6">comment #6</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=70983#c3">comment #3</a>)
> > (In reply to <a href="show_bug.cgi?id=70983#c2">comment #2</a>)
> > > @@ +433,5 @@
> > > > +  border = new AnnotBorderArray ();
> > > > +  border->setWidth (1.5);
> > > > +
> > > > +  annot = new AnnotGeometry (doc->doc, &pdf_rect, Annot::typeCircle);
> > > > +  annot->setBorder (border);
> > > 
> > > Why do we set a border by default? The BS entry is optional.
> > 
> > If we don't set the border, we get an "invisible" square/circle.

> This is a bug in poppler core then. Actually, it seems we are not exactly
> following the spec for borders, I'll fix it.</span >

I've just sent a few patches to the list that should fix this problem. With
those patches we don't need to set any border width at all to have square and
circle annotations, so for now I would not expose set_border_width, because
it's not needed and we might expose borders differently (using a
PopplerAnnotBorder object or something like that).</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>