Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

screen_icon Deprecated since version 3.8 #234

Closed
hamdan-mahran opened this issue Oct 11, 2014 · 4 comments
Closed

screen_icon Deprecated since version 3.8 #234

hamdan-mahran opened this issue Oct 11, 2014 · 4 comments

Comments

@hamdan-mahran
Copy link

Hi :)
my theme rejected because some of errors with Theme-Check plugin .
one of these error say that :

REQUIRED: screen_icon() found in the file theme-admin/plugins/class-tgm-plugin-activation.php. Deprecated since version 3.8.
Line 391: screen_icon( apply_filters( 'tgmpa_default_screen_icon', 'themes' ) );
Line 1585: screen_icon( apply_filters( 'tgmpa_default_screen_icon', 'themes' ) );

How can i fix that ?
Thanks

@denisbosire
Copy link

I just commented these two lines out, as far as I can see, they don't affect functionality

@hamdan-mahran
Copy link
Author

Thanks denisbosire :)
I commented these lines and the error gone
Thanks very much

@parkmap
Copy link

parkmap commented Oct 14, 2014

Thanks

@shivapoudel
Copy link
Contributor

Anyone Submit the PR :)

GaryJones pushed a commit that referenced this issue Apr 22, 2015
Manually re-applied to develop branch due to merge conflict after applying to master branch.

props @NateWr, see #244, fixes #234
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants