Skip to content

Wrong positioning of the search marks #85

Open

Description

@kenneth488

Hi KurtCode

I have the library with different PDF files, and it does often place the search marks wrong. In my fork, there is a PDF file called amazon-dynamo-sosp2007.pdf. Searching for the string "re" will show you that the position of the marks a sometimes calculated wrong, since they are misplaced. The same goes for the string "in" and probably others.

Is this something you might be able to fix?

Have a great day

Regards,

Kenneth

Activity

  1. viezel commented on Jun 9, 2015

    @viezel

    +1 could be nice to get sorted out.

  2. kenneth488 commented on Jun 11, 2015

    @kenneth488
    Author

    I have almost fixed the issue now, so the general solution works. Will commit and push today.

  3. kenneth488 commented on Jun 12, 2015

    @kenneth488
    Author

    I will commit it as soon as all the bugs are fixed! Are working on it atm! :)

  4. kenneth488 commented on Jun 12, 2015

    @kenneth488
    Author

    I will commit the fixes soon. I also implemented support for newline word searching, so we can mark search words spanning more than one line.

    When I commit, you can fork it and implement your fixes.

  5. nattapon21 commented on Jun 13, 2015

    @nattapon21

    .....

    2015-06-12 16:18 GMT+07:00 TryMyBest notifications@github.com:

    Ok. Will check your new version. Nice work guy! :)

    —
    Reply to this email directly or view it on GitHub
    #85 (comment).

  6. kenneth488 commented on Jun 13, 2015

    @kenneth488
    Author

    The commit is up. Now remember that there might be more bugs. If so, post them. I am still working on making it a lot better.

  7. kenneth488 commented on Jun 14, 2015

    @kenneth488
    Author

    Yes just saw that. Though that is only in the test project. It does work when you give it the correct page size. The way Kitten sets up a page, is pretty weird. It works fine otherwise:
    http://postimg.org/image/f5py9l2d7/
    http://postimg.org/image/ksmb724vv/

    I will see if I can fix it in Kitten.

  8. kenneth488 commented on Jun 14, 2015

    @kenneth488
    Author

    What you need to pass to the convertPDFRectToViewRect method is the frame of the userSpace PDF page aka. the visible pdf page you are viewing. Anything else will fail.

  9. timeflying commented on Aug 1, 2016

    @timeflying
  10. timeflying commented on Aug 1, 2016

    @timeflying

    I use the app preview which embedded in mac os . It works great on the same pdf.
    screen shot 2016-08-01 at 3 52 49 pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions