Project

General

Profile

Bug #5038

Cropfunktion til billede på sektion

Added by Gitte Barlach 8 months ago. Updated 24 days ago.

Status:
Unable to reproduce (Closed)
Priority:
Normal
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Forsiden, Inspiration - Sektioner og temaer

Description

Som følge af muligheden for at fremhæve en sektion på forsiden og dermed mulighed for at tilknytte et billede til en sektion, er det ønskeligt at få en cropfunktion sat ind på /taxonomy/term/-- /-- ved siden af Content image.

Målet må være at billedet så vidt muligt kan komme til at passe til visningen på forsiden. 


Files

Skærmbillede 2021-03-15 kl. 20.54.45.png (76.2 KB) Skærmbillede 2021-03-15 kl. 20.54.45.png Jeg kan ikke se at der skulle være en cropfunktion Gitte Barlach, 03/15/2021 08:58 PM
Skærmbillede 2021-04-28 kl. 21.32.19.png (260 KB) Skærmbillede 2021-04-28 kl. 21.32.19.png billedet på Sektion er croppet; billedet på Sektion2 er ikke Gitte Barlach, 04/28/2021 09:36 PM
Skærmbillede 2021-04-28 kl. 21.31.38.png (3.21 MB) Skærmbillede 2021-04-28 kl. 21.31.38.png Her udvælger jeg det udsnit af billedet, jeg gerne vil have vist på Sektion 2 Gitte Barlach, 04/28/2021 09:36 PM
Skærmbillede 2021-08-18 kl. 13.30.20.png (423 KB) Skærmbillede 2021-08-18 kl. 13.30.20.png Gitte Barlach, 08/18/2021 01:32 PM
Skærmbillede 2021-08-18 kl. 13.30.26.png (388 KB) Skærmbillede 2021-08-18 kl. 13.30.26.png Gitte Barlach, 08/18/2021 01:32 PM
Skærmbillede 2021-08-18 kl. 13.31.19.png (377 KB) Skærmbillede 2021-08-18 kl. 13.31.19.png Gitte Barlach, 08/18/2021 01:32 PM
#1

Updated by Thomas Hansen 8 months ago

  • Status changed from Ready for development to Needs code review
#2

Updated by Gitte Barlach 8 months ago

  • Assignee changed from Thomas Hansen to Jørgen Nielsen
#3

Updated by Gitte Barlach 8 months ago

  • Target version changed from Release 32 - Sektioner og Nodelists to Release 33 - Bugfixes
#4

Updated by Jørgen Nielsen 8 months ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

reviewet og godkendt

#5

Updated by Gitte Barlach 6 months ago

Testet på platform.sh
Testcase: cropfunktion er sat ind på /taxonomy/term/-- /-- ved siden af Content image.

Jeg kan desværre ikke se at der skulle være nogen cropfunktion. 

#6

Updated by Gitte Barlach 6 months ago

  • Description updated (diff)
#7

Updated by Gitte Barlach 6 months ago

  • Status changed from Reviewed to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Thomas Hansen
#8

Updated by Thomas Hansen 6 months ago

Så tillader jeg mig at spørge: er updb blevet kørt? Jeg har lige gjort det lokalt, og efter updb så er den der.

#9

Updated by Gitte Barlach 6 months ago

ja det skulle den være jvf. 
 

  # We run deploy hook after your application has been deployed and started.
  deploy: |
    set -e
    cd public
    drush -y updatedb
#10

Updated by Thomas Hansen 6 months ago

Ah, var for hurtig på aftrækkeren, har lige kigget i git historikken. Du skal prøve med en frisk installation. :-)

Tilføjelsen af crop funktion ved upgrade blev implementeret i #4241. Den her tilføjer at crop også bliver sat på nyinstallationer.

 

#11

Updated by Gitte Barlach 6 months ago

Hej Thomas. tak for info, er det dermed pull/1706 jeg skal teste?

 

#12

Updated by Thomas Hansen 6 months ago

...det forklarer selvfølgelig ikke hvorfor updb ikke virker på platform.sh i det her tilfælde... Men den blev testet til at virke i #4241

@gitte

Den er jo testet og merged?

#13

Updated by Thomas Hansen 6 months ago

Du har selv screenshottet at det virker: https://platform.dandigbib.org/attachments/5448

#14

Updated by Gitte Barlach 6 months ago

  • Status changed from Reviewed - Needs info/rework to Reviewed
  • Assignee changed from Thomas Hansen to Gitte Barlach

tak for opklaringen! 
Jeg sætter denne til "reviewed".

#15

Updated by Kasper Garnæs 5 months ago

  • Status changed from Reviewed to Integrationstest (Versioner)
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

#16

Updated by Kasper Garnæs 5 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach
#17

Updated by Gitte Barlach 5 months ago

Testet på upgrade-fbs med med 7.x-6.3.0-beta1
Verificeret at der er crop funktion til billede, der tilknyttes en sektion.

Men - skal jeg se på funktionaliteten må jeg indrømme jeg ikke helt forstår den. 
Se vedlagte eksmepler hvor jeg har taget udgangspunkt i helt det samme billede. På den ene sektion har jeg anvendt crop, og udvalgt et bestemt område af billedet, og gemt. Der vises et lille billede øverst til højre i crop-funktionen; tænkte det måtte væreet slags preview. På den anden har jeg ikke. anvendt crop.  Visningen på forsiden ser dog ens ud for mig at se. Skulle der ikke være forskel?

#18

Updated by Gitte Barlach 5 months ago

  • Status changed from Integrationstest (Versioner) to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Ninna Rasmussen
#19

Updated by Ninna Rasmussen 5 months ago

  • Assignee changed from Ninna Rasmussen to Thomas Hansen
  • Target version changed from Release 33 - Bugfixes to Release 34 - Bugfixes

Skal sendes tilbage til reload - jf. Gittes kommentar - ifb. rel. 34.

#20

Updated by Thomas Hansen about 1 month ago

@gitte

OK, jeg forstår heller ikke noget. På #4241 skrev du:

Testet på upgrade-fbs med 6.2.0-beta3
- man kan nu bskære det billede, der skal repræsentere en sektion. Godkendt.

Jeg har lige prøvet på min lokale der er en relativt frisk installation af 6.3.2 (jeg kan ikke se at der er sket noget i det område siden 6.3.0-beta), og der fungerer det fint.

Så jeg skal lige have nogen flere detaljer... Er det en frisk installation eller en upgrade?

#21

Updated by Gitte Barlach about 1 month ago

Hej Thomas

Det drejede sig om min kommentar 17 i nærværende sag, hvor jeg under min test ikke kunne se, at crop havde nogen effekt, uanset at jeg faktisk udvalgte et bestemt udsnit af et billede. 
MEN - jeg har netop gentestet på https://upgrade-fbs.ddbcms.dk/ med 6.3.2, og kan ikke genskabe fejlen. 

Jeg bemærker at cropfunktionen kun giver mening på billeder på de to første sektioner man vil fremhæve på forsiden; Resultatet bliver ikke godt hvis man cropper et billede på en sektion, der ligger som nummer 3 eller senere. (se vedlagte eksempler). Men det tænker jeg ikke vi skal gøre noget ved nu, medmindre det er meget let at fixe (set i lyset af NEXT)

#22

Updated by Thomas Hansen about 1 month ago

Mystisk.. Virker fint her (alle 3 billeder er udsnit):

#23

Updated by Gitte Barlach 29 days ago

  • Status changed from Reviewed - Needs info/rework to Unable to reproduce (Closed)
  • Assignee changed from Thomas Hansen to Gitte Barlach

hmm, ja, dine tests ser rigtige ud. Jeg kan ikke gennemskue hvorfor det nogle gange ikke virker for mig. Men da jeg faktisk ikke kan genskabe den oprindelige fejl længere, vil jeg foreslå at lukke sagen. 

Also available in: Atom PDF