Comments (7)
It looks like the first tr has no closing tag in the original source. Could you please try what happens if you properly close the tag?
from emogrifier.
fixed and tried my code - still the same problem (copy & paste mistake....)
from emogrifier.
Could you please try to reduce the HTML and CSS to the absolute minimal test case that still produces the problem? Or, even better, pour it into a (failing) unit test?
from emogrifier.
Well, the test side http://www.pelagodesign.com/sidecar/emogrifier/ cannot reproduce the error (anymore), I will close this ticket - but we had to fix it with commenting out https://github.com/jjriv/emogrifier/blob/master/Classes/Emogrifier.php#L348 - L355. Otherwise we have empty tr in our mails...
from emogrifier.
Thanks for the feedback. Now I'm confused: Is the problem still present in the latest version available from GitHub, or do we still need to fix it?
from emogrifier.
:D I cannot reproduce it anymore on the emogrifier testsite - but we still have the problem on our system. Perhaps because of a sideeffect in the html - it was not possible for me to trac the html down to a minimal version.... And I have no time for that atm - we have our workaround (like discribed above) and the customer is happy :)
Perhaps I will have time sooner or later to recheck.... but not at the moment.
from emogrifier.
Okay. If you have more information, please feel free to reopen this ticket. Thanks for reporting the issue!
from emogrifier.
Related Issues (20)
- Resolve Psalm 5.0 warnings
- HTTP 403 error during GitHub action HOT 10
- EOL only when necessary HOT 2
- Is it possible to mark future PHP versions as untested but available in Composer?
- Re-establish support for PHP 7.2 HOT 3
- Add a branch without an upper PHP version limit HOT 4
- Psalm warning re closure passed to usort HOT 5
- Disable HTML formatting by default HOT 8
- Upgrade to phpunit-data-provider 3.0.0 HOT 1
- Psalm errors on Windows with PHIVE Phar HOT 5
- Return values from preg_replace* are not checked
- Inconsistency in dev tool versions selected for CI build HOT 3
- Breaking change in 7.1 HOT 10
- Add support for Symfony 7
- Drop support for unsupported Symfony versions HOT 2
- Switch from the abandoned `squizlabs/php_codesniffer` to `phpcsstandards/php_codesniffer` HOT 2
- Fix composer-normalize on CI HOT 3
- Add readme caveat re URL escaping
- Avoid generating a `composer.lock`
- Use the development PHP INI on CI
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from emogrifier.