Widelands Website

Deleting from the outbox will redirect to inbox

Bug #1714162 reported by GunChleoc
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Widelands Website
Fix Released
Low
Unassigned

Bug Description

After deleting a message from the outbox, the inbox is shown instead of the outbox.

Related branches

lp:~widelands-dev/widelands-website/pm_menus_overhaul
GunChleoc: Approve
Diff: 257 lines (+103/-24)
12 files modified
mainpage/templatetags/all_users.py (+12/-0)
media/css/messages.css (+47/-0)
media/css/search.css (+1/-1)
media/js/jquery-ui.min.js (+10/-10)
pip_requirements.txt (+2/-1)
templates/base.html (+1/-0)
templates/django_messages/base.html (+6/-4)
templates/django_messages/compose.html (+13/-1)
templates/django_messages/inbox.html (+1/-1)
templates/django_messages/outbox.html (+1/-1)
templates/django_messages/trash.html (+9/-3)
templates/search/search.html (+0/-2)
kaputtnik (franku)
Changed in widelands-website:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
kaputtnik (franku) wrote :

The workflow in the messages view for deleting messages is not good anyway. IMHO it would be good to have checkboxes instead of the 'x' which triggers a reload every time. So a user can check the messages he want to delete and click a button 'Delete marked messages'.

Revision history for this message
GunChleoc (gunchleoc) wrote :

+1

kaputtnik (franku)
Changed in widelands-website:
status: Confirmed → In Progress
assignee: nobody → kaputtnik (franku)
Revision history for this message
kaputtnik (franku) wrote :

The current state is:

- Clicking on 'delete' in in/outbox will not redirect to the inbox anymore, instead the current view (in/outbox) will be used further.

- Clicking on 'undelete' in the trash will redirect either to the inbox or the outbox, depending on where the initial message was deleted.

I didn't implemented checkboxes right now, because this needs some changes which should be done by the app developer of django-messages. I think the current state is also a good solution.

Revision history for this message
GunChleoc (gunchleoc) wrote :

I agree.

kaputtnik (franku)
Changed in widelands-website:
status: In Progress → Fix Released
assignee: kaputtnik (franku) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.