search

mebis-lp / mod_learningmap

GitHub Repository for mod_learningmap
https://moodle.org/plugins/mod_learningmap
GNU General Public License v3.0
13 stars 6 forks source link

Changing image size fails #68

Open bselmshorn opened 1 year ago

bselmshorn commented 1 year ago

Changing size of the used image fails, because of heigth and width is saved somewhere in the plugin and therefore not changeable.

sh-csg commented 1 year ago

Hi @bselmshorn , thanks for reporting this issue! Could you please provide some information to reproduce the problem?

bselmshorn commented 1 year ago

Sollte man sich nach dem Erstellen einer Lernlandkarte dazu entschließen das Bild der Karte gegen eines mit anderer Skalierung zu tauschen, bleibt die "Original"-Größe erhalten. Das bleibt auch so, wenn man die geänderte Landkarte abgespeichert. Der einzige Workaround ist es, die Landkarte komplett zu löschen. Es wäre schön, wenn man irgendwo die Skalierung eingeben kann oder zumindest das Bild zurücksetzen könnte.

bselmshorn commented 1 year ago

das "closed" ist ein Fehl-Klick, den ich nicht mehr ändern kann :(

sh-csg commented 1 year ago

Welche Version verwendest du denn?

bselmshorn commented 1 year ago

Lernlandkarte 0.9.2 von 2023020701

Von: Stefan Hanauska @.***> Gesendet: Dienstag, 7. März 2023 07:47 An: mebis-lp/mod_learningmap Cc: Conrad Krödel; State change Betreff: {Disarmed} Re: [mebis-lp/mod_learningmap] Changing image size fails (Issue #68)

Welche Version verwendest du denn?

— Reply to this email directly, view it on GitHubhttps://github.com/mebis-lp/mod_learningmap/issues/68#issuecomment-1457632128, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKB6Y3RMDEIUFUV2U66JLMDW23KXLANCNFSM6AAAAAAVQ4B7KA. You are receiving this because you modified the open/close state.Message ID: @.***>

[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/mebis-lp/mod_learningmap/issues/68#issuecomment-1457632128", "url": "https://github.com/mebis-lp/mod_learningmap/issues/68#issuecomment-1457632128", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

sh-csg commented 1 year ago

Ich kann das geschilderte Problem bisher leider nicht reproduzieren. In welchem Browser tritt es denn auf?

bselmshorn commented 1 year ago

Die Lernlandkarte wird anscheinend (ungefragt) immer auf Seitenbreite skaliert. Wenn das Hintergrund nun eher lang als breit ist, geht die Karte ggf. über mehrere Bildschirmseiten, was dazu führt, dass die schöne übersichtliche Darstellung des Lernpfades verloren geht, da gescrollt werden muss. Sehr kleine Hintergrundbilder werden großskaliert, so dass die Qualität darunter leidet. Wünschenswert wäre eine Breite-Höhe-Einstelloption, zumindest fakultativ. Ansonsten gilt der Tipp, nur Hintergrundbilder zu verwenden, die rechteckig sind mit einer Breite, die größer ist als die Bildhöhe.

Nach längerem Ausprobieren bin ich nun eher der Meinung, dass der Beitrag hier mehr ein Feature-Request als ein Bugreport sein müsste.

sh-csg commented 1 year ago

Ah, jetzt verstehe ich, wie die Anfrage gemeint ist. Die derzeitige Variante ist tatsächlich so beabsichtigt, ggf. können wir dieses Feature aber implementieren.