8000 Fixes/use svg icons by jrief · Pull Request #6455 · django-cms/django-cms · GitHub
[go: up one dir, main page]

Skip to content

Fixes/use svg icons #6455

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

Closed
wants to merge 36 commits into from
Closed

Conversation

jrief
Copy link
Contributor
@jrief jrief commented Jul 23, 2018

Summary

Fixes #
If CMS_PERMISSION = True the inline admin of ALL PERMISSIONS renders icons. Since Django-1.9 these icons have been replaced by SVG variants, so use them.

Documentation checklist

  • I have updated CHANGELOG.txt if appropriate
  • [*] I have updated the release notes document if appropriate, with:
    • general notes
    • [*] bug-fixes
    • improvements/new features
    • backwards-incompatible changes
    • required upgrade steps
    • names of contributors
  • I have updated other documentation
  • I have added my name to the AUTHORS file
  • This PR's documentation has been approved by Daniele Procida

czpython and others added 30 commits April 16, 2018 10:34
…m model to form (django-cms#6362)

Increased meta description length restriction to 320 characters
* Improved toolbar tutorial
* Fixed typo
* Improved URLs explanation
* Made minor improvements to tutorial
* Update apphooks.rst

You need to add *Application instance name* by hand to a page when you add an ``app_name`` #hopeitsavessomeoneelsethetrouble

* Changed as per czpython's suggestions

While still trying to convey that we're fixing the same problem than the previous paragraphs refers to (and that *Application instance name* only appears after setting ``app_name`` )
* Updated apphooks how-to and introduction

[ci only docs]
@coveralls
Copy link
coveralls commented Jul 23, 2018

Coverage Status

Coverage remained the same at 78.189% when pulling a998160 on jrief:fixes/use-svg-icons into 9d19af9 on divio:release/3.5.x.

Copy link
Contributor
@czpython czpython left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jrief
3.5 still supports 1.8
Can you push your changes to the develop branch?
Also please add an entry to CHANGELOG and remove the one from upgrade notes

@czpython czpython self-assigned this Jul 24, 2018
@czpython czpython added this to the 3.6.0 milestone Jul 24, 2018
jrief added 4 commits July 25, 2018 00:54
# Conflicts:
#	CHANGELOG.txt
#	cms/appresolver.py
#	cms/tests/test_fixture_loading.py
#	cms/tests/test_permissions.py
#	docs/how_to/apphooks.rst
@jrief jrief closed this Jul 24, 2018
@jrief jrief reopened this Jul 24, 2018
@jrief jrief closed this Jul 24, 2018
@jrief jrief mentioned this pull request Jul 25, 2018
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0