-
Notifications
You must be signed in to change notification settings - Fork 673
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
WARNING: Replace fill-available to stretch, because spec had been changed #3989
Comments
Same for me `WARNING in ./node_modules/froala-editor/css/froala_editor.pkgd.min.css (./node_modules/css-loader??ref--5-2!./node_modules/postcss-loader/src??postcss0!./node_modules/froala-editor/css/froala_editor.pkgd.min.css) (7:52021) Replace fill-available to stretch, because spec had been changed |
Same for me... |
Same, can this please get fixed? |
Same for me... |
I had contact with Froala today and they will fix the issue in v3.2.4 what will be released in a few months. |
@arwinvdv Thanks for the information ;) |
Is there a way to work around this until a fix is provided? |
normally npm is not blocking the script. it's just a warning. If you want to fix it, you can explore the package, do a search and replace and replace "fill-available" to "stretch". But it's only temporary and it's better waiting a real fix. |
Even I am also getting the same warning in console. |
Thanks for the update, having the same issue. |
gracias! |
I've the same issue. Any update on this? |
Same issue :) |
having same problem here. i have been searching for an hour and its frustrating. i hope it fixed very soon. |
+1, seeing same issue here as well |
Hi Everyone, as noted earlier, it looks like they will fix this in v3.2.4, however if you want to get rid of the error you can open the froala_editor.pkgd.min.css file in your node_modules and do a search and replace Search: fill-available Save the file and now your builds wont fail. Warning: as you are editing a vendor node library file, any npm update or re-install of the package will require these changes gain until they fix in the future update. |
Is there a fix that does not require modifying node_modules?... This is not a fix that could be done in any situation but locally, considering node_modules aren't committed (at least for my project...) |
I have the same problem!! It would be nice for them to release the fix soon!! |
Same problem here. Is there a fix to surpress thiss message in the meantime? |
I suppressed the message by add string-replace-loader in webpack config rules. // webpack.config.js
module.exports = {
// ...
module: {
rules: [
{
test: /froala_editor\.pkgd\.min\.css$/,
loader: 'string-replace-loader',
options: {
search: /fill-available/g,
replace: 'stretch'
}
}
]
}
} |
works for me! thank you a lot @iicdii 👏👏👏 |
Hi folks! I saw 3.2.4 came out yesterday, and admittedly I've been tracking it closely for the fix to this issue per the comment below:
I didn't see it in the release notes: and updating, I see the issue persists. Can someone from Froala kindly advise? For those of us who don't want to manually edit our third party packages, it would be great to get an update! |
Agreed, @Taskle ! This has been a HUGE thorn in our sides...I know it may seem like a small thing, but we have a clean console policy, and the time it takes to sort past those warnings every time we see something real that needs to be addressed adds up. Plus it's just a giant eyesore in something we take pride in. This needs to be fixed! If Froala doesn't respond to this issue soon, we'll certainly be looking towards another rich text editor and not renewing our subscription. I was so happy to see that 3.2.4 dropped, and you can imagine my disappointment when I upgraded, hoping to finally see a clean console and.... |
@EthanRBrown, how many options do we have out there? :) |
I just checked the new version and as said above, warnings still exists. So I looked to the mail they send me and I see that I wrote it wrong here. |
So since this still seems to be an issue (🙄)... here is how we handled it in our app... I just brought in webpack-filter-warnings-plugin, and applied it to my webpack config like so: const baseConfig = {
name: pkg.name,
entry: [paths.appIndexJs, 'whatwg-fetch'],
output: {
path: path.resolve(__dirname, '../dist'),
publicPath: '/',
},
module: {
rules: [
configureFontLoader(),
configureImageLoader(),
configureIconLoader(),
],
},
resolve: resolveConfig,
plugins: [
new WebpackNotifierPlugin({
title: 'Webpack',
excludeWarnings: true,
alwaysNotify: true,
}),
new webpack.ProvidePlugin({
FroalaEditor: 'froala_editor.min.js/froala_editor.pkgd.min.js',
}),
/*
@HACK: Froala currently has a warning that postcss throws, and the only solution from the author is basically to edit node_modules ~until 3.2.4 is released~...
Well since that won't work for us (node_modules is not committed), I have decided to filter the warnings so it doesn't make our logs ginormous between now
and when it is released.
Relevant Issue: https://github.com/froala/wysiwyg-editor/issues/3989
@FUTURE: remove me + corresponding plugin ~when froala-editor is upgraded >=3.2.4~ (also remember to update storybook/main.js)!
*/
new FilterWarningsPlugin({
exclude: /Replace fill-available to stretch/,
}),
],
}; 💨 Poof, no more warnings 😄 |
For those of us that are using React's CRA and not ejected to use custom webpack configs, sounds like this is only an option if we either eject or use CRACO. Sounds like we're stuck waiting until 3.4... if folks who are also using CRA have thoughts on a workaround without a ton of work/changes/additions, feel free to share! Indeed this has been the only warning lingering in our app for months, so unfortunate that it will continue to linger. |
@ansh-les I will. At this point, I'm tempted to go with CKEditor 5, though I haven't investigated the options yet. Quill is also very interesting, more of a new kid on the block. |
This is also causing us to reassess our choice of froala. Does anyone know why it is taking so long to resolve? |
This is VERY annoying indeed. For the time being I have added the following script to my
However, this needs to be executed after every |
@kgish Nice one-liner, worth placing it inside a |
Glad I'm not alone here. Just upgraded to 3.2.6 after dealing with the crazy licensing issues.. Now console is full of warnings. Is this still slated for 3.4? that seems like a long way out. Will be on Angular 12 by then. |
You can also have a node solution to support all systems (Windows): Create a file called const fs = require('fs');
const froalaMinCss = 'node_modules/froala-editor/css/froala_editor.pkgd.min.css';
fs.readFile(froalaMinCss, 'utf8', function (err, data) {
if (err) {
throw err;
}
const result = data.replace(/fill-available/g, 'stretch');
fs.writeFile(froalaMinCss, result, 'utf8', function (e) {
if (e) throw e;
});
}); Essentially, it does the same thing as the Then, add it to your Finally, run |
The issue has already been solved. The fix will be available in version 3.3.3 (mid-March at the latest). Thanks |
Experiencing the same issue :) |
March has ended. not fixed yet. |
It's 30.03, UTC 10AM. They still have time. |
I see a PR open since Aug, 2020 that fixes this issue: The issue and the PR are not linked though. Can we please get it merged? |
March has ended. not fixed yet ( 2 ) |
Same issue here! @dejanmartinovic do you know when the version 3.3.3 will be available? |
I think, nobody from support reads this threads. |
The fix for this issue will be released in the next editor release which is expected to be released by the first week of May. |
which year??? |
Snark aside, thanks for the update |
any updates ? |
This week? |
This day? |
This month? |
This year? |
At this life or after reincarnation?
…On 14.05.2021 19:08, jaymathew wrote:
This year?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#3989 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAH2MSDNHNRKDFU5CNLK4LTTNVDHXANCNFSM4PD2RWRA>.
|
I don't know where @ilyaskarim got his information, but I just mailed Froala support and they say:
Edit: Apparently if you ask when the next release is, they are assuming a major release. |
Awesome! Now Froala will charge us to solve this issue, since we all have V3 licensing… great way to force us tu buy a new license.🤮 |
Finally fixed in https://froala.com/wysiwyg-editor/changelog/#3.2.7 |
/{YOUR_PROJECT PATH}/node_modules/react-pure-modal/dist/react-pure-modal.min.css (go to this file path , then follow these steps) -- first search for the word 'fill-available ' |
is ok? |
Expected behavior.
Not have warnings like this when building node dependency with a project.
Actual behavior.
Steps to reproduce the problem.
Build a project that has this project as one of its npm dependencies
Editor version.
3.1.1
OS.
Windows 10 1909
Browser.
Irrelevant
The text was updated successfully, but these errors were encountered: