Project

General

Profile

Project activity #30175

closed

Tezza: documenten in preview kunnen draaien (rotate)

Added by Juri Balradj 5 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
P4 Normal
Category:
-
Target version:
Start date:
10/12/2024
Due date:
% Done:

0%

Estimated time:

Description

Het toevoegen van een draai-actie aan de toolbar boven het document die het html-element draait op basis van css.
  • Toevoegen van een viewer toolbarAction
  • Toevoegen van action & effect
  • Toevoegen method om rotate class toe te voegen aan DOM per draaiactie
  • Aanpassen CSS om te reageren op action

Zie comment https://support.contezza.nl/issues/30014#note-3
video https://support.contezza.nl/attachments/27340

Zie analyse #30014 voor meer info


Files

clipboard-202501061505-emuak.png (35.8 KB) clipboard-202501061505-emuak.png Diego Mirandola, 06/01/2025 03:05 PM
Actions #2

Updated by Juri Balradj 5 months ago

  • Tracker changed from Service request to Project activity
  • Parent task set to #29758
Actions #3

Updated by Juri Balradj 5 months ago

  • Description updated (diff)
Actions #4

Updated by Juri Balradj 5 months ago

  • Description updated (diff)
Actions #5

Updated by Juri Balradj 5 months ago

  • Assignee set to Contezza Development
Actions #6

Updated by Tahir Malik 5 months ago

  • Assignee changed from Contezza Development to Maaike Bommerson
Actions #8

Updated by Maaike Bommerson 4 months ago

  • Status changed from New to In Progress
Actions #9

Updated by Maaike Bommerson 4 months ago

  • Status changed from In Progress to Ready in Dev
  • Assignee changed from Maaike Bommerson to Diego Mirandola

Graag testen in branch feature/viewer/rotate/#30175

Actions #10

Updated by Diego Mirandola 4 months ago

Opmerkingen over de code:
  • graag de path naar de nieuwe library toevoegen in tsconfig en deze path gebruiken in de imports
  • er zijn props in de action definition die niet worden gebruikt, graag weghalen
Functionele opmerkingen:
  • de icon toont een pijltje tegen de klok in, de rotatie is met de klok mee, graag het effect aanpassen met rotaties van -90, -180, -270
  • de action is beschikbaar op jpg of txt documenten en in dit geval werkt niet, ik zou de actie alleen beschikbaar maken als het document mimetype pdf heeft, graag met Juri controleren als dit inderdaad voldoende zou zijn. De klant vraagt deze functionaliteit expliciet voor pdf bestanden, dus dit zou voldoende moeten zijn.
  • de action module wordt in core geïmporteerd maar de assets worden alleen in tezza geïmporteerd. Dit betekent dat deze functionaliteit werkt niet in gemutr-tezza en dat alle apps een foutmelding gaan tonen over de ontbrekende assets. Ik zou deze module alleen in gemutr-tezza importeren (maar je kunt dit beter Tahir navragen)
Actions #11

Updated by Diego Mirandola 4 months ago

  • Status changed from Ready in Dev to In Progress
  • Assignee changed from Diego Mirandola to Maaike Bommerson
  • Target version changed from Tezza Backlog to Contezza Development 2025.01
Actions #12

Updated by Tahir Malik 4 months ago

Dit mag generiek beschikbaar gemaakt worden, ik zie geen breaking impact op de rest van de implementaties.

Actions #13

Updated by Maaike Bommerson 4 months ago

  • Status changed from In Progress to Ready in Dev
  • Assignee changed from Maaike Bommerson to Diego Mirandola

Graag opnieuw testen.
Het is nu in alle tezza apps geimporteerd en zichtbaar voor office apps aangezien het hier wel werkt

Actions #14

Updated by Tahir Malik 4 months ago

  • Target version changed from Contezza Development 2025.01 to Tezza 2025.01
Actions #15

Updated by Diego Mirandola 4 months ago

  • de action is beschikbaar op jpg of txt documenten en in dit geval werkt niet, ik zou de actie alleen beschikbaar maken als het document mimetype pdf heeft, graag met Juri controleren als dit inderdaad voldoende zou zijn. De klant vraagt deze functionaliteit expliciet voor pdf bestanden, dus dit zou voldoende moeten zijn.

Dit probleem speelt nog steeds bij json bestanden.

  • de action module wordt in core geïmporteerd maar de assets worden alleen in tezza geïmporteerd. Dit betekent dat deze functionaliteit werkt niet in

gemutr-tezza en dat alle apps een foutmelding gaan tonen over de ontbrekende assets. Ik zou deze module alleen in gemutr-tezza importeren (maar je kunt dit beter Tahir navragen)

Dit is niet gefixed, bijvoorbeeld in contezza-migration:

Actions #16

Updated by Maaike Bommerson 4 months ago

  • Status changed from In Progress to Ready in Dev
  • Assignee changed from Maaike Bommerson to Diego Mirandola

Nu verplaatst naar de tezzamodule en de assets toegevoegd in de nodige apps. Actie nu alleen zichtbaar bij pdf
Kan je nog eens testen?

Actions #17

Updated by Diego Mirandola 4 months ago

  • Status changed from Ready in Dev to Test approved
  • Assignee changed from Diego Mirandola to Maaike Bommerson

Test is akkoord.

Actions #18

Updated by Maaike Bommerson 4 months ago

  • Status changed from Test approved to Resolved

Dit zit in contezza apps versie 3.18.2

Actions #19

Updated by Juri Balradj 4 months ago · Edited

  • Status changed from Resolved to Feedback
Actions #20

Updated by Juri Balradj 4 months ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF