After dragging field rows on the manage display page, an Ajax call is made to refresh the rows. This is not needed any longer.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bceyssens created an issue. See original summary.

bceyssens’s picture

Status: Active » Needs review
FileSize
1.06 KB

This patch removes the AJAXRefreshRows functionality.

In addition I came across a related issue together with the Field group module. When a field is dragged from within a field group to the first position in the region, the settings are not saved. This is also an issue when this patch is not applied.

dxx’s picture

FileSize
142.29 KB

This patch hide the system message at the top (indicate not saved changes)? Because this message displayed before ajax refresh move all contents after (rows) to bottom and make confusion to user (see here)...

aspilicious’s picture

@dxx this patch shpuld do the opposite, leave the message in place. Did you clear the caches?

dxx’s picture

@aspilicious Yes, all is good with this patch ;)

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

  • aspilicious committed 4a2fca8 on 8.x-2.x authored by bceyssens
    Issue #2612426 by bceyssens: Ajax refresh after row drag is not needed
    
aspilicious’s picture

Status: Reviewed & tested by the community » Fixed

Thnx all! :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.