Jump to content

horst

PW-Moderators
  • Posts

    4,088
  • Joined

  • Last visited

  • Days Won

    88

Everything posted by horst

  1. Looks really good! Many thanks! Can‘t wait to arrive into my office and setup my list and add all to my starter profile. Really great! ?
  2. @Robin S really really nice would be, if we can load a custom list with names AND optionally individual download URLs, so that we also can include modules that are not in the official directory. Priority: If a module is selected that has a individual download URL, paste it into the "Add Module From URL" field. What do you think?
  3. Hey @kixe, one question before trying this out: Is it possible to see image thumbnails to select from? I have a use case where I need centralized stored icons and a select field for reference, that shows image thumbnails. Can I do that with your module?
  4. You can post an issue on github, as Ryan may not see this here in the post. But he will see it on Github!
  5. I don't get any letters, but I had deleted my FF-Sync Account because I got doomed by their expired Cert for AddOns. First it deleted my local AddOns and then "uploaded" it to my Sync-Account, what in fact deleted all AddOns there too. For me it was the drop that broke the camel's back. I switched to Opera and there are now only two points of 20+ left which I haven't solved to my satisfaction in Opera yet. In some regards Opera even is a better tool then FF. I really liked it after two weeks using. It seems that this has caused, among other things, a great loss of trust or resentment, which may have led to reduced user numbers. I still use the FF to test websites in the FF, but for nothing else.
  6. This is not an easy question, and so also isn't the answer. Personally I would prefer Git submodules as it would be less effort for users who doesn't have either Git nor Composer understanding. Also Git is very tiny action to install and running, composer seems to take more effort. So, for users who have both capabilities, I also would go with option 3, the composer dependencies. My conclusion is, it depends on your main target group.
  7. @Robin S Many thanks for this. I will try it soon!
  8. @huseyin Welcome to ProcessWire and many thanks for sharing this project with us. Its impressive and well done. ? Paylaştığın için teşekkür ederim
  9. Why do you not include type hidden fields into your forms where you add identifiers? $validFormIdentifier = ['aaaaaaaa', 'bbbbbbb', 'ccccccc']; $myIdentifiedForm = $sanitizer->text($input->post->hiddenidentifier); if(!in_array($myIdentifiedForm, $validFormIdentifier)) { // invalid formsubmission, do not process further }
  10. @psy You can set webpQuality to lower values what will result in smaller filesizes. Examples:
  11. The only thing what may occure in very rare cases is, that another module in the same PW installation also uses the TwitterOAuth lib, stored in a different place. To avoid conflicts, one can add a conditional check before including it, using the PHP function class_exists(). <?PHP namespace ProcessWire; if(!class_exists("Abraham\TwitterOAuth\TwitterOAuth") { /* load the TwitterOAuth library from my Module folder */ require_once("./twitteroauth/autoload.php"); } use Abraham\TwitterOAuth\TwitterOAuth; Also one may prefer require_once() over require(), if there is a chance that the class / class file is loaded twice.
  12. Hi all users of the croppable image 3 fieldtype. I'm currently working on the module, or better I'm reworking the module to support webp copys like the core image field does. Unfortunately this will break backwards compatibility! ? My questions are, how should we handle this? Until now the crop variation has this sort of names: basename.ext becomes basename.-suffix.ext But with the new version, it looks like regular image variation files with a -suffix: basename.ext becomes basename.100x200-suffix.ext My current thoughts are along that lines: I create a new named module: CroppableImage4, that supports the same API as the CAI3 version plus an alternative method like getCrop4("suffix"), and I add some checking, when the CAI3 is installed / active, it only hooks the plus method, not the regular one. What do you think? Any suggestions, ideas, wishes? And how can we savely update from CAI3 to CAI4, for example with existing websites containing tons of images ❓ PS: Besides that, the new module now supports individual $options as array or selector strings. It generates 0x48 thumbs and webp copies by auto generation and by manually creating a crop. It removes all variations on every changes: changing a single image, changing or removing settings. It has input fields in modules config for webpAdd and webpQuality, so you can handle it different from the $config->imageSizerSettings, and you now can add individual options with every API call, like with the core image field: $pageimage->getCrop("suffix", $options)->url, or $pageimage->getCrop("suffix", $options)->webp->url. And some more minor tweaks.
      • 2
      • Like
  13. Hohoho, a 3000 x 3000 PNG, for what do you need this? You generate a 1000 x 1000 variation? If you use photoshop jpeg quality 12 you also has lossless master images. My favor would be to have LZW-Tiffs as master, = Lossless compression and a format that cannot wrongly used for web output. ?
  14. Yep, and I / we don't like version hunting for all the format issues in Imagick or GD. ?
  15. Besides the broken Imagesupport for PNG8 I want to say: "Uploading ImageOptim optimized images are ONLY for output use as is(!), NOT for serving as master image from what one should generate new variations. ? " Everytime when I read something like this, my photographer heart hurts and somewhere a kitten dies, I believe. ?? ?
  16. Ah, @ryan has added support for PNG8 to the Imagick engine, what wasn't before. He also asked me why this wasn't added and I was unsure if I simply forgot to add it. But now when I see this, I think I had tested it and removed it from the Imagick supported filetypes / subtypes, so that this images always were rendered by GD! until before the last Update. https://github.com/processwire/processwire/pull/141#issuecomment-495983196 So it seems it should be rolled back again, at least for PNG8. EDIT: I couldn't remember as it is (how many?) years ago as the Image Engine System was introduced. And yep, it would have been better to add some comments. ;-)
  17. You should try harder!
  18. The image rendering engines have gotten a method / check if and which supports webp creation. I think you best drop an issue at pw github, so that ryan gets knowledge about it. If I find time I try to check this too.
  19. Here is a first try with webp support: github webp-dev !! ATTENTION: DO NOT USE ON PRODUCTION SITES !! Maybe the cleaning up for unused variations is broken now. Haven't tested! Also the naming scheme of CAI3->getCrop("suffix") has changed from basename.-suffix.ext to basename.{width}x{height}-suffix.ext Would be fine if some of you have time to test and report back in regard of: removing no longer used variations, changing values of already defined crop settings, (and if previously used variations get deleted, etc) ... Currently there has to be defined true in site/config.php for $config->imageSizerOptions("webpAdd", true); Then you can use it like with core images: $image = $page->images->first->getCrop("suffix"); echo "<img src='{$image->url}' /> <img src='{$image->webp->url}' />";
  20. When I try it, I get something like: Notice: Undefined index: _width in \wire\core\Pageimage.php on line 1936 Notice: Undefined index: _height in \wire\core\Pageimage.php on line 1937 These are the new additions from Ryan that I'm currently have not checked how it works, but definetly, if width and height isn't known, the engines cannot create a new variation. I get this for image objects derived from CAI3 and with the original image too! Possibly a bug in the new pageimage additions, or a different use case that currently is not covered.
  21. A while ago I bookmarked this: https://css-tricks.com/flutter-googles-take-on-cross-platform/
  22. @ryan thank you very much for all the additions you implemented in such a short time on top of my github pullrequest. It seems that there currently nothing is left in regard of webp support in ProcessWire. But I already have one or two points for future updates to the image rendering engines, that should be implemented when all browsers will support webp and we don't need fallback jpegs or pngs anymore. ?
  23. @Lutz Heckelmann Oh, thanks a lot! Now I know what is missing. The column filedata is available in my local version, but is missing in the live version. Only difference between the upgrades from local and live is, that the local one was updated step by step since the last two years, as it is symlinked to my pw-dev-latest directory, but the live one was updated from 3.0.62 to 3.0.130 in one go. So, now I will alter the table to add this missing column. EDIT: Issue solved! ?
×
×
  • Create New...