Created attachment 28298 [details]
PDF file with annotations
has annotations that do not show with Evince 2.27 and Poppler 0.11
KaL on #evince says:
ok, I know what the problem is
and yes, it's a poppler issue
that annot doesn't have an appearance stream, poppler should provide default AP streams for such cases
*** Bug 24364 has been marked as a duplicate of this bug. ***
Created attachment 31662 [details]
Attaching the 48px icons , I had shown Kal on #evince.
He mentioned few problems on the evince side and was not sure if the annotations would work , do let me know if it works and I can get the rest of the icons done too.
Also , attaching the 24px versions of a couple of icons ,of the idea I had.
To use the square as the constant shape for the icon and symbols embedded in the square. That way the instant the user notices the shape , the user realizes it is an annotation. [This is just an idea , if the icons are desired similar to the acroread , that can be done too.]
A couple of screenshots showing how pdf_commenting_new.pdf looks with that icon:
Second screenshot doesn't look correct due to bug #25441 :-(
Created attachment 31779 [details] [review]
Set of patches
This set of patches adds appaearance streams for Text, Line and Gemoetry annotations. In the case of Text annotations it uses the icon provided by mac_v in this bug (we might want to modify it, but the code is mostly the same)
It can be tested with this document:
From a quick look the patch seems mostly ok though there are some indentation issues (lines that had spaces and you add tabs) also notes in page 3 seem too small (not sure you are tackling this already)
Want me to run reg testing with your changes? Or not still?
(In reply to comment #5)
> From a quick look the patch seems mostly ok though there are some indentation
> issues (lines that had spaces and you add tabs)
hmm, I'll review it, I hadn't noticed it.
> also notes in page 3 seem too
> small (not sure you are tackling this already)
Yes, it's because of bug #25441, not sure how to fix it, though.
> Want me to run reg testing with your changes? Or not still?
Not yet, I'm going to add more icons for Text annots and probably for file attachment annots too.
Thanks for reviewing :-)
Thanks to ulisse and mac_v we'll have more icons soon. I still don't know how to apply the color specified in the annotation dictionary, the spec only says it should be used to draw the annot background. We've done severals tests:
The important thing here is not the icon itself, but how we apply the annot color (yellow in this case). 1) and 5) are indeed the same approach but with different icons.
which one do you prefer?
Created attachment 31819 [details]
poppler svg icons
Ulisse and myself have completed the set of icons as requested by Kal.
Attaching the svg's to the bug report
(In reply to comment #7)
> Thanks to ulisse and mac_v we'll have more icons soon. I still don't know how
> to apply the color specified in the annotation dictionary, the spec only says
> it should be used to draw the annot background. We've done severals tests:
> 1.- http://people.freedesktop.org/~carlosgc/poppler-paperclip-note.png
> 2.- http://people.freedesktop.org/~carlosgc/poppler-paperclip-note2.png
> 3.- http://people.freedesktop.org/~carlosgc/poppler-paperclip-note3.png
> 4.- http://people.freedesktop.org/~carlosgc/poppler-paperclip2.png
> 5.- http://people.freedesktop.org/~carlosgc/poppler-paperclip3.png
> The important thing here is not the icon itself, but how we apply the annot
> color (yellow in this case). 1) and 5) are indeed the same approach but with
> different icons.
> which one do you prefer?
2.- http://people.freedesktop.org/~carlosgc/poppler-paperclip-note2.png looks good to me.
(In reply to comment #9)
> 2.- http://people.freedesktop.org/~carlosgc/poppler-paperclip-note2.png looks
> good to me.
Ulisse and myself really dont like option 2, for 2 reasons.
- It uses too much color and distracts the user from the real content of the page.
- If we are to use option 2 , then all the annotation icons wouldnt look consistent.
The current icons [attached in comment #8] will all use the color in the "frame" the corners(when specified) while the icons themselves would be grey and will look consistent. > http://imagebin.ca/view/B0p_8c.html
If the icons are to be done as in option 2 , we'd end up with the same loud colored icons of acroread , which are very distracting from the page contents.
(In reply to comment #10)
> (In reply to comment #9)
> > 2.- http://people.freedesktop.org/~carlosgc/poppler-paperclip-note2.png looks
> > good to me.
> Ulisse and myself really dont like option 2, for 2 reasons.
> - It uses too much color and distracts the user from the real content of the
> - If we are to use option 2 , then all the annotation icons wouldnt look
> The current icons [attached in comment #8] will all use the color in the
> "frame" the corners(when specified) while the icons themselves would be grey
> and will look consistent. > http://imagebin.ca/view/B0p_8c.html
> If the icons are to be done as in option 2 , we'd end up with the same loud
> colored icons of acroread , which are very distracting from the page contents.
yes,i hguess you are right.
comment #8 icons are better
You may or may not like how Adobe Reader draws the icons but the specification says how they should be drawn and not doing it the way the specification says because you think it looks ugly is not an option, sorry.
(In reply to comment #12)
> You may or may not like how Adobe Reader draws the icons but the specification
> says how they should be drawn and not doing it the way the specification says
> because you think it looks ugly is not an option, sorry.
Are you sure that the icons are specified to be done the exact same way?
The specs seem to only mention >
"C - array - The background of the annotation’s icon when closed"
But , strictly speaking the acroread icon's background isnt really colored.
Rather the color is applied within the icon. While the background is transparent.
It seems that the acroread icons were designed to only allow the use of color rather than sticking to the exact specs. IMO , we can take the same liberties too.
To me the specs are a little fuzzy , and too huge.
I might have missed the part where the icon design was specified.
Could you point the right page where the icon design is mentioned ? [we can correct the icons accordingly]
Well, I also prefer option two, but the spec doesn't say proposed icons are wrong.
Well, that's a discussion of terminology, but for me
is the only one that is painting the background as said on the spec
are using the color in a frame
are using the color in the pen
Created attachment 32041 [details]
Poppler icons with background color
Attaching the latest set of icons.
Created attachment 32073 [details] [review]
Updated set of patches
Here is the updated set of patches for current master using the new icons.
Created attachment 32120 [details]
Icons for tag and graph
Annot Icons for tag and graph
Created attachment 32124 [details]
Icons for mic and speaker
Annot Icons for mic and speaker
Created attachment 32150 [details] [review]
This patchset applies on top of the previous one and includes:
- Fix a memleak of previous patches
- Fix hightlight annots (see commit message) and provide appearance streams for TextMarkup
- Add Graph and Tag icons for FileAttachment
- Add speaker and mic icons for Sound
- Fail if QuadPoints is not present in TextMarkup since it's required
I've just pushed both patch sets. I leave the bug open becuase there are still annotations missing default appearance streams (Free Text annots, rubber stamp, ...)
*** Bug 36660 has been marked as a duplicate of this bug. ***
Was this ever merged?
I just filed a bug in Okular related to this.
This was 9 years ago but the ticket is still open and the patches still exist.
(In reply to Kevin Burton from comment #23)
> Was this ever merged?
Do you know how to read? comment #21 is *VERY* clear, isn't it?
Let's try to keep the comments constructive and technical so we can avoid arguments that will distract us from the common goal that we all share of improving annotations.
-- GitLab Migration Automatic Message --
This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.
You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/poppler/poppler/issues/522.