Skip to content

Drawing : Fixed emuToPixels (returns float and not int) #58

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

Merged
merged 3 commits into from
Feb 28, 2025

Conversation

Progi1984
Copy link
Member

No description provided.

@Progi1984 Progi1984 added the bug label Feb 28, 2025
@Progi1984 Progi1984 added this to the 1.0.5 milestone Feb 28, 2025
@Progi1984 Progi1984 merged commit 5a2eeb8 into PHPOffice:master Feb 28, 2025
20 checks passed
@Progi1984 Progi1984 deleted the emuToPixels branch February 28, 2025 12:39
@Brenneisen
Copy link

@Progi1984 After this update, we are getting errors in a PowerPoint file that was created and needs to be repaired when opened. Do you have any idea why this is happening? Thank you!

@Progi1984
Copy link
Member Author

@Brenneisen The fix is on the way : PHPOffice/PHPPresentation#856. Sorry for problems.

@Scheissy
Copy link

Scheissy commented Mar 4, 2025

hey @Progi1984!

I investigted the issue today and could fix it by changing a default value in the TextStyle constructor. In Line 55 the setIndent is called with -324900 / 9525. One line below the setMarginLeft is called with 342900 / 9525. If I change the value in setIndent to -342900 / 9525 I can open the created presentation without any issues.

@Progi1984
Copy link
Member Author

@Scheissy It's a BC Break on next version. Line Width will be in pixels, not in points. It's documented. 👍

@Scheissy
Copy link

Scheissy commented Mar 4, 2025

@Progi1984 I don't understand what the issue has to do with line width. I'm talking about this line. when you compare this line with the next line you can see that there might be a typo in the denominator: 324900 vs 342900.
If I use 342900 in both cases I don't get any issues.

@Progi1984
Copy link
Member Author

@Scheissy Fixed : PHPOffice/PHPPresentation#857

@Brenneisen
Copy link

Hi @Progi1984 can you already estimate when there will be a new release with these fixes?

@Brenneisen
Copy link

@Progi1984 Could you please make a release? It's great if the bugs are fixed, but this way we are forced to switch to "phpoffice/phppresentation": "dev-master" branch or use "phpoffice/common": "<1.0.5". The interval between releases is very long. Although many PRs have already been merged. Is there a reason why the next release 1.2.0 should fix so many more issues?

@PowerKiKi
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants