-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show baselines #18
base: master
Are you sure you want to change the base?
Show baselines #18
Conversation
Fixes #17 |
Note: 9cfcd81 fixes mouse scroll wheel behaviour. I never got ctrl+scroll zooming to work before (and was surprised to find an implementation). It seems that only the I'll promptly rewrite (and rebase) that part of the PR if you find anything objectionable here. |
Strangely, if I export the app as runnable JAR, I get a different behaviour for mouse wheel scrolling than running directly from Eclipse! When running from Eclipse, ctrl+scroll correctly zooms in or out, depending on the direction. But when running from the runnable JAR, |
Another strange thing is that in my newly built version, #15 re-appears. My understanding was that this was fixed by ad75ac2 and PRImA-Research-Lab/prima-core-libs@28db6c0. Since I have not changed the build settings, So perhaps @chris1010010 you used some additional changes for the 1.4.06 binary release that you have not published yet? |
Plus the prebuilt version can deal with TIFF images, whereas the configured build cannot. |
I have merge in the baseline code |
Splendid. So how about that part:
|
(happens for me e.g. for 16-bit TIFF or grayscale TIFF)
Do you import some additional library for your GH release builds? |
No, it's all the same SWT as far as I'm aware. |
I see. Then perhaps this is a change between OpenJDK 8 and 11? Or do you use a different JDK8 for your prebuilds?
Understood. Did mouse wheel based horizontal/vertical scrolling and zooming work for you so far on any platform? (I've only ever tested Linux 64 bit with OpenJDK 8, and there neither horizontal scrolling nor zooming worked.) But mind that extra trip of runnable JAR vs running in Eclipse... |
This is the bare minimum. You might want to add a new button to switch this on or off. Perhaps Aletheia already has an icon for that?
Also, I had to import
org.primaresearch.dla.page.layout.physical.text.impl.TextLine
, because there is no interface which givesget_Baseline()
.