๐Ÿ‡บ๐Ÿ‡ธUnited States @nm63282

Seattle, WA
Account created on 16 April 2021, about 4 years ago
#

Recent comments

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Thanks again for calling this out.
I've made progress and have drafted a pull request but it'll likely be next week before I finishing testing and updates to go with it.
PR: https://git.drupalcode.org/project/decorative_images/-/merge_requests/

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Thanks for opening this. Iโ€™ll begin on it shortly. The approach will likely be updating the code to hook the file widget instead of image widget but, can stay pretty narrow by targeting the alt field. Will share an update once Iโ€™ve made progress.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Greetings tijsdeboeck,

Accessibility is a complicated area with many important voices, and hard work. Iโ€™ll reach out to the authors of decorative_images_widget and hope for a good discussion.

I welcome feedback and criticism altho, not if it prohibits innovation or prohibits people from coming to the table. A simple search for โ€œcookie complianceโ€ demonstrates there may be many, many solutions to a need in Drupal.

This module was developed in response to real-world needsโ€”including work with professional disability testers, in university environments, and with enterprise content governance. It addresses a need not available in core, an image may require good alternative text OR may be decorative, hence a module. Thanks for raising this.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA
๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA
๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

@Vishal,

Can we move this forward?

Thanks

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

I've attached a patch that address the issues and is pretty minimal. But recurring_events can be heavily relied on and by sites with lots of events traffic so it'd be helpful getting review from another maintainer.

Here a summary of the changes, which are within recurring_Events_registration:

  1. Change hook `eventseries_insert` to `eventinstance_insert`
  2. Change criteria in `eventseries_update` to simple yes/ no
  3. Do not empty `reminder_sent`

I'll mark this as Needs review.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Thanks your clear write up and for the report @phillamb168. I can reproduce the same, although it sounds like you found workarounds.

Here's my write up to help drafting a pull request. I should be able to draft something shortly.

1) Clicking Add instance on a series creates an instance with `reminder_date` empty
2) When `reminder_date` changes `reminder_sent` is emptied
3) If series dates changes but reminder options did not, instances are recreated without a `reminder_date` (It's expected Instances are recreated when series dates change)

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

I was just testing and see it on a core field--

To repro:
1. Disable smart_date, multiple_fields_remove_button.
2. Create two new fields:
2.a. Date and time (Date, multiple)
2.b. Date and time (Timestamp, multiple)

Add multiple values to each field on a node, then save it.
Edit the node and try removing values.

Result:
1. Can remove all Date values
2. Cannot remove all Timestamp values

I could be wrong but, looks like core (maybe multiple_fields_remove_button too) need an update. I'll try and look more.

Versions: Drupal version : 10.4.5

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

1.x branch updated; marking Ready for review.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Thank you for review Vishal.

I've made those updates:
- Branch `develop` deleted;
- info.yml updated
- .module comment updated.

These changes appear in tag 1.0.1 [ https://git.drupalcode.org/project/decorative_images/-/tags/1.0.1 ]

We have release branch (not tag) `1.x`. What numbering do you recommend if tag `1.2.0` already exists?

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

ll66382 โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

https://www.drupal.org/project/drupal/issues/3347021#comment-15967244 ๐Ÿ’ฌ CKEditor 5 Media Embed alignment options does not work Closed: cannot reproduce And @megan_m 's solution worked for me:

- Limit allowed HTML tags and correct faulty HTML
- Correct faulty and chopped off HTML

Also Embed media near the bottom so Media links are not removed per https://www.drupal.org/project/drupal/issues/3075527#comment-15604529 ๐Ÿ› Adding links around embedded media through CKEditor might lead to invalid/complex markup when rendered Active

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

I have recurring_events on a couple of sites now although, I have to make sure I have databases for them. I have contributed patches and code for recurring_events. I'd be glad to help out here and there. Could you go ahead and add me? If yes I'll try watching the queue to add help or comments where possible.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

ll66382 โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Has there been any discussion of adding a "Decorative" checkbox to images?

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Could I be added as a contributor/ co-maintainer helper of this guide? I'm routinely using API's as a part of a large portfolio of sites. I'd be glad to verify or make suggestions to the guide's materials when time allows. 

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

nm63282 โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

nm63282 โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

nm63282 โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Move UI tips/ instructions to bottom. 

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Improve page title.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Cleanup. 

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Improve code documentation. 

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Improve testing instructions.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Add source code.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Improve clarity.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

Here is a sample plugin for above.

Note it would break complex permissions, for example block rules below or beside allow rules.

web/modules/custom/zzz/src/Plugin/ImcePlugin/ZzzImceFolders.php


namespace Drupal\zzz\Plugin\ImcePlugin;


use Drupal\Core\Session\AccountProxyInterface;
use Drupal\imce\Imce;
use Drupal\imce\ImcePluginBase;

/**
 * Custom IMCE folder perms merge.
 *
 * @ImcePlugin(
 *   id = "zzz_imce_folders",
 *   label = "IMCE ZZZ Folders",
 *   weight = 10,
 * )
 *
 * Explain.
 */
class ZzzImceFolders extends ImcePluginBase {

  /**
   * {@inheritdoc}
   */
  public function processUserConf(array &$conf, AccountProxyInterface $user) {

    // Skip administrators.
    if ($user->hasPermission("administer imce")) {
      return;
    }

    // Skip admin roles.
    if (array_intersect($user->getRoles(), [
      'administrator',
      'custom_client_admin',
    ])) {
      return;
    }

    // Get imce settings.
    $settings = \Drupal::config('imce.settings');
    $imce_roles = $settings->get('roles_profiles');

    // Get user roles, from more permissive to less permissive.
    $user_roles = array_reverse(array_flip($user->getRoles()));
    $user_imce_roles = array_keys(array_intersect_key($imce_roles, $user_roles));

    // Get storage.
    $scheme = \Drupal::config('system.file')->get('default_scheme');
    $storage = \Drupal::entityTypeManager()->getStorage('imce_profile');

    // Get folders for each role.
    foreach ($user_imce_roles as $id) {
      if ($profile = $storage->load($imce_roles[$id][$scheme])) {
        if ($folders = $profile->getConf('folders', [])) {
          $processed = Imce::processUserFolders($folders, $user);
          $conf['folders'][$id] = $processed;
        }
      }
    }
  }

}
๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

This still occurs on 3.0.0-beta8.

Oddly, the patch still applies too.

looks like the issue may be multiple dialogs getting inserted.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA

nifsight โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States nm63282 Seattle, WA
Production build 0.71.5 2024