Lesterhuis-Training-en-Consultancy / moodle-block-user_favorites

Moodle plugin - marking pages as favorites
GNU General Public License v3.0
1 stars 4 forks source link

Adding sorting bookmarks #1

Closed luukverhoeven closed 1 year ago

gemguardian commented 1 year ago

@luukverhoeven could you please let me know how much time it would be to add this feature to M4.1/php8

gemguardian commented 1 year ago
Comment from Freshservice:
Freshservice: 153 linked successfully.
luukverhoeven commented 1 year ago

@gemguardian How should the sorting work? Is this drag-drop javascript or with arrows icons moving the positions?

gemguardian commented 1 year ago

@luukverhoeven let's keep it simple, arrows icons moving the positions

luukverhoeven commented 1 year ago

The estimated time required to add this feature is three hours.

gemguardian commented 1 year ago

@luukverhoeven approved

luukverhoeven commented 1 year ago

@gemguardian Aanpassing staat klaar in https://github.com/Lesterhuis-Training-en-Consultancy/moodle-block-user_favorites/tree/8677fr957-sorting-bookmarks

Kun je controleren of dit naar wens werkt?

2023-03-02_10-40-51 (1)

gemguardian commented 1 year ago

@luukverhoeven klopt het dat als je nu een pagina toegevoegd, dat je eerst je pagina moet refreshen vóórdat je kan verschuiven?

luukverhoeven commented 1 year ago

Dit issue is opgelost in de laatste versie https://github.com/Lesterhuis-Training-en-Consultancy/moodle-block-user_favorites/tree/8677fr957-sorting-bookmarks

Kun je dit bevestigen?

gemguardian commented 1 year ago

@luukverhoeven nadat ik deze versie updaten kreeg ik problemen met het ladeblok in M4.1.1+ (build 202303030) thema Boost

Het ladeblok wil niet meer open of sluiten op de pagina waar het staat De console geeft de onderstaande melding

image

luukverhoeven commented 1 year ago

@gemguardian Dit heb ik zojuist bekeken, maar kon het niet reproduceren. In de error lijkt het te gaan om fastnav ipv user_favorites

Zijn er bepaalde acties die je uitvoert? Welke versies heb je geïnstalleerd van deze twee plug-ins?

image image image

gemguardian commented 1 year ago

@luukverhoeven ik had deze versies die jij heb hebt geïnstalleerd. Ik heb Fast Navigation verwijderd, cache clear gedaan maar ik krijg nog steeds dezelfde melding

image

luukverhoeven commented 1 year ago

@gemguardian Zie hierbij mijn schermopname. Gek genoeg kom bij mij geen tooltip error voorbij. https://app.screencast.com/H7kvIItTv8ovh

Als ik zoek in de broncode van deze plug-in, zie ik geen tooltip voorbij komen.

image

gemguardian commented 1 year ago

heel apart. Omdat je tooltip aangaf, dacht ik misschien is het de aanpassing in VSF dat loopt te vervelen. Maar die gede-installeerd en dat lost het nog niet op.

Enige dat ik zo kan bedenken is een conflict met een andere plugin...

gemguardian commented 1 year ago

Okay het is een plugin conflict . want op een andere 4.1 installatie heb ik het issue niet (ook niet op verschillende servers). Nu wordt het dus uitpuzzelen welke plugin het is.

gemguardian commented 1 year ago

Ok dit was heel apart - bij een ander account werkte het wel en toen ik dus een startpagina reset deed was het opgelost

gemguardian commented 1 year ago

@luukverhoeven akkoord kan door naar MAIN

luukverhoeven commented 1 year ago

@gemguardian Doorgezet naar de hoofdbranch en gepubliceerd