Have you had this feeling, when you have Features and Diff enabled, an overridden feature, you have the grey "Overridden" link in the state column of your features list, and click on it just to be redirected to the default tab, and having to click the "Review Overrides" tab to see the diff?

I think if the "Review Overrides" tab is available -ergo the Diff module enabled-, the natural destination for the "Overridden" link in the features list is this tab. Attached is a patch to alter that link.

Please leave a comment if you find this addition useful.

Comments

pere orga’s picture

Status: Needs review » Needs work

Cool.

One minor thing, though. "overridden" is misspelled in href_overriden variable.

kaldimar’s picture

Issue summary: View changes
StatusFileSize
new1.32 KB
kaldimar’s picture

StatusFileSize
new1.32 KB

Noted and corrected

kaldimar’s picture

Title: Overriden link should point to diff tab » Overridden link should point to diff tab
kaldimar’s picture

Status: Needs work » Needs review
pere orga’s picture

Status: Needs review » Reviewed & tested by the community

Works and looks good to me

kaldimar’s picture

I've been using this patch on a couple production sites and it's working fine. If further testers can confirm it's usefulness please comment here. I believe it's a nice time saver for those projects doing heavy use of Features.

hefox’s picture

Status: Reviewed & tested by the community » Fixed

thanks

  • hefox committed 5a9dbd0 on 7.x-2.x authored by kaldimar
    Issue #2336555 by kaldimar: Overridden link should point to diff tab
    

  • hefox committed 5a9dbd0 on 8.x-3.x authored by kaldimar
    Issue #2336555 by kaldimar: Overridden link should point to diff tab
    

Status: Fixed » Closed (fixed)

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