Problem/Motivation

I noticed that the deprecation warning about defaultTheme links not to the change record, but to the issue. I think it would be more helpful to link to the change record, which has the relevant information in a condensed form, rather than the issue which loads slower because it has over 130 comments, and would require searching for the recommendations.

Three nodes relate to this deprecation, the change record: https://www.drupal.org/node/3083055
and these two issues
https://www.drupal.org/node/2352949
and https://www.drupal.org/project/drupal/issues/3082655

Proposed resolution

Change the node id 2352949 to 3083055 in the deprecation warning.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3095598-2.patch1.17 KBoknate

Comments

oknate created an issue. See original summary.

oknate’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new1.17 KB
lauriii’s picture

Status: Needs review » Reviewed & tested by the community

I agree that it would be better to link to the change record. Confirmed that the link works so this looks good. ✨

  • catch committed 2fe3b82 on 9.0.x
    Issue #3095598 by oknate: defaultTheme deprecation warning should link...

  • catch committed b8c7b53 on 8.9.x
    Issue #3095598 by oknate: defaultTheme deprecation warning should link...

  • catch committed 80b144b on 8.8.x
    Issue #3095598 by oknate: defaultTheme deprecation warning should link...
catch’s picture

Category: Feature request » Task
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.0.x/8.9.x/8.8.x, thanks!

oknate’s picture

Excellent, thanks!

Status: Fixed » Closed (fixed)

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