Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Template tags: check_path_active does not account for i18n #8

Closed
Wtower opened this issue Nov 29, 2015 · 1 comment
Closed

Template tags: check_path_active does not account for i18n #8

Wtower opened this issue Nov 29, 2015 · 1 comment
Labels

Comments

@Wtower
Copy link
Owner

Wtower commented Nov 29, 2015

  • node_path = '/'
  • request_path = '/el/'
@Wtower
Copy link
Owner Author

Wtower commented Nov 29, 2015

This can be overcome when using node.full_path instead of node.path when checking if menu item is active in block_menu_header.html.

Updating default template.

@Wtower Wtower added the bug label Nov 29, 2015
@Wtower Wtower closed this as completed in f62718c Nov 30, 2015
Wtower added a commit that referenced this issue Dec 9, 2015
What's new

- Added feature: remove uploaded file (and any image styles-thumbnails) when the relevant record is removed.
- Improved documentation
- Improvements
- Upgraded to Django 1.9

All changes

- Fix #11
- Added help texts to models with reference to documentation
- Added images to the project
- Added more blocks in ``base.html``.
- Fixes #8
- Added feature: remove uploaded file (and any image styles-thumbnails) when the relevant record is removed.
- Added pre-delete signal.
- It appears that multiple pre-delete functions with decorators do not get called, so merged function with
  previous ``delete_guardian_group_perms``.
- Using added function in media.py to search for all files with the same name, below the path, in order to trace
  all thumbnails and then remove all.
- Updated tests to cover new feature.
- Minor update in readme.
- Modified code that returns the path of an uploaded media file be OS-agnostic
- Improved test to allow different title in admin (still containing "administration")
- Improved documentation
- Added class to index.html to improve navigation in inspector
- Added reference to django-ninecms-starter on readme
- Improved documentation
Wtower added a commit that referenced this issue Dec 12, 2015
Corrected active trail functions in template tags and reverted the changes in the menu template.
Created small utility function to get url without language.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant