I have tested it now with a pure online nightly Joomla 4 and it has exactly the same issue.

• Create modules in Position "breadcrumbs" or so if you have no demo data installed.
• Create a modul "Articles Category".
• Go to "Extensions manager" and deactivate "Articles Category" extension.
• Go back to modules manager.
• Fiddle like described in previous #30798 (comment)

Additional find: One can't reorder the deactivated module with Drag&Drop.

@dgrammatiko Can you please advise changes to the following code to only add the click event to rows containing a checkbox input? Thank you!

  class JMultiSelect {
    constructor(formElement) {
      this.tableEl = document.querySelector(formElement);

      if (this.tableEl) {
        this.boxes = [].slice.call(this.tableEl.querySelectorAll('input[type=checkbox]'));
        this.rows = [].slice.call(document.querySelectorAll('tr[class^="row"]'));
        this.checkallToggle = document.querySelector('[name="checkall-toggle"]');

        this.onCheckallToggleClick = this.onCheckallToggleClick.bind(this);
        this.onRowClick = this.onRowClick.bind(this);

        if (this.checkallToggle) {
          this.checkallToggle.addEventListener('click', this.onCheckallToggleClick);
        }

        if (this.rows.length) {
          this.rows.forEach((row) => {
            row.addEventListener('click', this.onRowClick);
          });
        }
      }
    }

<tr class="row0" data-draggable-group="none">
  <td class="text-center">
  </td>
</tr>
<tr class="row1" data-draggable-group="none">
  <td class="text-center">
   <label for="cb3"><span class="sr-only">Select </span></label><input autocomplete="off" type="checkbox" id="cb3" name="cid[]" value="130" onclick="Joomla.isChecked(this.checked);">
  </td>
</tr>

@Quy sure, the problem is that for disabled rows the checkbox needs to render with a disabled attribute instead of skipping it. Basically the conditional here

Added the disabled attribute to checkbox. Clicking the checkbox does not toggle it, however, clicking the row/check all items checkbox toggles the checkbox.

you need something like:

      if (this.boxes[currentCheckBox].disabled) {
        return;
      }

Almost there. The disabled checkbox can still be toggled using the Check All Items checkbox at the top. Thank you!

This appears to be resolved now as the last issue reported is no longer valid and this should be closed

If you say so I believe it ;-)

This is reproducible.

Install Testing Sample Data.
Go to System > Manage > Extensions
Search and disable Wrapper - Site.
Go to System > Manage > Site Modules
Click Wrapper row.
The checkbox below it is checked.

Thanks @Quy for reopening! I have confused this with another selection failure pr (menu assignment)

What am I doing wrong then

@brianteeman try to select the next module after the disabled one

ah - it selects the next two. got it now

See demo of problem here in the NOTE
#34238 (comment)

Why don't we simply do

						<td class="text-center">
							<?php echo HTMLHelper::_('grid.id', $i, $item->id, false, 'cid', 'cb', $item->title); ?>
						</td>

imho, it does not matter if we can tick the disabled item checkbox and use Actions as it does not change its disabled status.

thats not a good solution as it provides a negative user experience which was the entire point in the checkbox being removed in the first place.

TBH, I'm more surprised when the checkbox is removed at all, because then I have to think and guess and have to make a connection: no checkbox? why? mhh, because it's disabled (says the icon at least).

If there would be an disabled checkbox with information why it's disabled, then I don't have to think myself...

thats not a good solution as it provides a negative user experience which was the entire point in the checkbox being removed in the first place.

Certainly less negative than what we have now. The module can anyway be edited although it is disabled. Taking off the checkbox makes no sense.

Concerning the explanation, I guess using a real tooltip as proposed here would do the job.#34238 (comment)

thats not a good solution as it provides a negative user experience which was the entire point in the checkbox being removed in the first place.

Whoever removed the checkbox for the disabled items did a very poor job as they broke at least 3 scripts: drag and drop, multiselect and select all. Can it be done without a checkbox for disabled items? Yes but all 3 scripts need to be adjusted and honestly, it's far easier to bringback the checkbox as I already guided @Quy in my comments above

thats not a good solution as it provides a negative user experience which was the entire point in the checkbox being removed in the first place.

Whoever removed the checkbox for the disabled items did a very poor job as they broke at least 3 scripts: drag and drop, multiselect and select all. Can it be done without a checkbox for disabled items? Yes but all 3 scripts need to be adjusted and honestly, it's far easier to bringback the checkbox as I already guided @Quy in my comments above

• Deletion of this entry is also not possible anymore...

The use of <?php if ($item->enabled > 0) : ?> for the checkbox was introduced in J3.5.0
#8020

If a component is disabled, we remove the link in the menu to this component. No Tooltip, no ghost componetnt - disabled means disabled. Why not hide disabled module instances from the list?

Why not hide disabled module instances from the list?

Actually, I wouldn't expect the list all modules to display only rows for modules that are enabled. In fact, I would expect when a module row indicated that the module is disabled and I enable (publish) that row then the module will be enabled automatically instead of having to go to the extensions to enable the module and come back to publish the row. It seems that we're making things hard for no reason

Also if you hide rows like that in case I uninstall a module I have no way to remove any remaining related data in the database

I see this: A disabled module is not in the select screen (com_modules&view=select).
Because it is disabled and may not be used - maybe old PHP version or whatever the reason is for the administrator.
If another user could publish it in the same way as a unpublished module in (com_modules&view=modules), this would be contrary to the intention of disabling the module.

In any case, re-adding the checkbox and disabling it so that no action is possible would at least solve the problem of the missing checkbox

In any case, re-adding the checkbox and disabling it so that no action is possible would at least solve the problem of the missing checkbox

Except, as @bembelimen rightfully remarked, that one can't delete the instance of the module concerned via the admin UI.
Very different from not displaying the menu link to the component.

Just saying: In j3 disabled module items are not shown in the list. It is true that the items remain as ghost in the database.

The difference in J3 is that a blocked module can't be disabled.
Therefore the test should be done with a non-blocked module.

In j3 disabled module items are not shown in the list.

In my case they are shown:

Yep. Same for Most Read because it is not blocked.

J3 list of protected modules

I have added checkbox, please test #34273

closing it, because PR