misc: make card linked to page update page title (#74073) #201

Merged
fpeters merged 1 commits from wip/74073-card-page-title into main 2023-12-18 13:20:07 +01:00
Owner
No description provided.
fpeters added 1 commit 2023-12-12 14:24:06 +01:00
gitea/combo/pipeline/head There was a failure building this commit Details
0d66a3ff5e
misc: make card linked to page update page title (#74073)
fpeters force-pushed wip/74073-card-page-title from 0d66a3ff5e to b13172de80 2023-12-12 14:33:18 +01:00 Compare
fpeters force-pushed wip/74073-card-page-title from b13172de80 to 91e276e034 2023-12-12 15:08:11 +01:00 Compare
fpeters reviewed 2023-12-12 15:26:32 +01:00
@ -1686,0 +1687,4 @@
card_data.get('digest') or card_data.get('digests')
):
# card linked to page, set attribute to have card title in page <title>
page_title_from_cell = card_data.get('digest')
Author
Owner

Dans le cas qui nous intéresse, page liée à un modèle de fiche, on cherche un titre (soit le digest, général, soit digests['default'], soit digests['custom-view:XXX']).

Dans le cas qui nous intéresse, page liée à un modèle de fiche, on cherche un titre (soit le digest, général, soit digests['default'], soit digests['custom-view:XXX']).
@ -1686,0 +1696,4 @@
if card_data.get('digests', {}).get(digest_name):
page_title_from_cell = card_data['digests'][digest_name]
if page_title_from_cell:
custom_context['request'].page_title_from_cell = page_title_from_cell
Author
Owner

C'est plutôt moche pour transporter l'information tout du long l'endroit facile c'est l'objet request.

C'est plutôt moche pour transporter l'information tout du long l'endroit facile c'est l'objet request.
@ -34,0 +36,4 @@
if (document.original_title === undefined) {
document.original_title = document.title
}
document.title = page_title + ' | ' + document.original_title;
Author
Owner

en js, on met à jour.

en js, on met à jour.
@ -203,0 +202,4 @@
response = HttpResponse(response_content, content_type='text/html')
if hasattr(request, 'page_title_from_cell'):
# cell request a change to page title, pass info in response header
response.headers['X-Page-Title'] = request.page_title_from_cell
Author
Owner

Lors du rendu ajax de la cellule, on met l'info dans un entête custom.

Lors du rendu ajax de la cellule, on met l'info dans un entête custom.
@ -619,0 +622,4 @@
response_content = loader.render_to_string(template_name, ctx, request)
if hasattr(request, 'page_title_from_cell'):
# a synchronous cell requested a change to page <title>, do it.
response_content = response_content.replace('<title>', f'<title>{request.page_title_from_cell} | ', 1)
Author
Owner

Pour le cas du rendu synchrone, on va modifier direct le <title>.

En passant je note que la manière dont c'est affiché se base sur ce qu'on a aujourd'hui dans publik-base-theme, à savoir "titre de page | titre de site" (i.e. | comme séparateur, le plus spécifique à gauche).

Pour le cas du rendu synchrone, on va modifier direct le `<title>`. En passant je note que la manière dont c'est affiché se base sur ce qu'on a aujourd'hui dans publik-base-theme, à savoir "titre de page | titre de site" (i.e. | comme séparateur, le plus spécifique à gauche).
fpeters changed title from WIP: misc: make card linked to page update page title (#74073) to misc: make card linked to page update page title (#74073) 2023-12-12 15:26:37 +01:00
lguerin approved these changes 2023-12-18 08:56:52 +01:00
fpeters merged commit 20a37687d6 into main 2023-12-18 13:20:07 +01:00
fpeters deleted branch wip/74073-card-page-title 2023-12-18 13:20:07 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#201
No description provided.