πŸ‡ΊπŸ‡ΈUnited States @adrian_s_m

Account created on 18 December 2018, almost 6 years ago
#

Recent comments

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I'm giving this a shot as part of MidCamp 2024:

Enables your site to search within file attachments. Think of it as giving your site the ability to read and search inside PDFs, documents, and more via a few extraction methods.

Thanks!

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I like #6 πŸ“Œ Create a short description for the Paragraphs Editor Enhancements project Needs review the best as it encompasses most of the functionality of the module.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I like the logo, but the file should be named logo.png as that is the requirement. Thanks!

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

The logo in #1 meets the requirements in terms of size, but the file should be named logo.png.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

This logo can work, but one of the requirements is that the files is named logo.png.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

Reviewing this as part of MidCamp 2024, #7 πŸ“Œ Facets - Add a short description RTBC works and meets the criteria.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

This can work and meets the criteria, but it would be great to have a few other options so that we have better representation.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

For me #9 πŸ“Œ File metadata manager - Add a short description Needs review works and meets the criteria.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I think #20 meets the criteria and uses the original logo used by the module maintainers, so I would assume they would want to use that instead of changing it.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

Uploading new file patch, since I set the boolean value to a string previously. This is needed for 10.1.x

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

We also tested this and works as expected. We are also interested in a stable release asap. Thank you!

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

We tested this internally with a multi-value media filed and works as expected. Thank you!

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

Proposed resolution:

Twig Field Value module helps the frontenders to get partial data from Drupal field render arrays. It gives them more control over the output without using preprocess functions.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I would remove this part "It can be used to create single or multiple field values such as content." from the proposed resolution since there is no mention of that in the readme.md or the description of the module here Twig Field Value β†’ .

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I think #2 would be enough, not sure if icons fall under "media" as proposed by @brianbrarian though.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

Approved πŸ‘!

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I agree with #5, "utility" would be great to add.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

#2 meets the criteria, but he also raises a good point, if this module should be included or be part of this initiative at all?

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

#2 seems correct to me as well.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

#2 meets the required criteria, thank you @hebl.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I believe #4 makes sense to me as well. The maintainer states on his blog that it's for spam protection Introducing the Honeypot form spam protection module for Drupal

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I agree with number #4 seems more future proof.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I personally like option3 with the blueish background gradient, but It's bigger than the requirements in size(KB) and 460*460 instead of 512*512.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

The logo at #4 looks great and meets the required criteria.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

Can one of the requirements be that logos stick to a certain color palette or a few colors approved by the community or maybe something that matches the Olivero theme or the default theme? I'm not a designer but it would be nice to have some consistency when looking at ll the logos. As of right now they are all over the place in term of colors/background/no background. Some designer input and guidance would be appreciated.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

This looks good and meets the required criteria.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

This looks good and mets the required criteria.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

This looks good and meets the criteria.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

I also like the GTM-logo-1 variant and I confirm that it's meets the criteria.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

"Node Field" sounds better to me, since the initial comment.

πŸ‡ΊπŸ‡ΈUnited States adrian_s_m

We used this NodeField plugin and works as expected.πŸ‘

Production build 0.71.5 2024