Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nevergone created an issue. See original summary.

nevergone’s picture

Status: Active » Needs review
FileSize
436 bytes

Please review this patch.

riddhi.addweb’s picture

@nevergone Thanks! For the patch it works well, But found one minor typo in the tooltip.
PFA Screenshot.

riddhi.addweb’s picture

Status: Needs review » Needs work
FileSize
86.61 KB
nevergone’s picture

@Jigar.addweb: Thanks this review!

nevergone’s picture

Status: Needs work » Needs review
riddhi.addweb’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
60.21 KB

Thanks! @nevergone for Your quick reply.

Your Patch Works well for me.

Thanks.

  • Berdir committed 97876ca on 8.x-1.x authored by nevergone
    Issue #2837544 by nevergone, Jigar.addweb: add menu items
    
Berdir’s picture

Status: Reviewed & tested by the community » Fixed

Menu links for /admin/content/* are IMHO just a weird workaround to have them show up in the admin toolbar, that shouldn't be required but fine, lets add that. Would have vetoed that if it would point to the view route, as that is risky when the view is renamed/deleted/disabled, then everything breaks.

Committed.

Status: Fixed » Closed (fixed)

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