Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dokumenthanterare: Koppling karta - kapitel fungerar inte m.m #279

Closed
ingvar-uddevalla opened this issue Sep 3, 2019 · 27 comments
Closed
Assignees
Milestone

Comments

@ingvar-uddevalla
Copy link
Contributor

Så här är det:
Dokumenthanterare (eller som vi i Uddevalla och i ÖP-sammanhang kallar den Kapitelindelning) är generellt sett bra och Kungsbacka m.fl har ju lagt till en utskriftsmöjlighet m.m som verkar lovande. Dock kvarstår en del irriterande buggar/avvikelser.
Kopplingen från karta till kapitel fungerar inte. Inga lagerkort (cards) visas här heller (jfr issue #276 för Sök).

image

Så här vill vi ha det:
Vi vill kunna lägga en länk till relaterat kapitel i Dokumenthanteraren från Lagerhanteraren, från lagerkorten/cards (t ex genom ikonen ”T”) och om möjligt med en länk från attribut-popupen (infoklicken).
När man klickar på Karta och vill visa de lager som t ex representerar kapitlet Energiproduktion, så ska tidigare tända lager fortsatt vara tända. Annars förstår nog inte användarna varför de tidigare lända lagren släcktes. Bäst är förstås om man som administratör kan ställa in om hur denna rensning ska utföras (allt eller delar av).
Innehållsförteckningen behöver en logisk följd. Man ska kunna ta sig vidare (och tillbaka) utan att behöva gå tillbaka till hela innehållsförteckningen. Behövs ex. knapp till nästa kapitel.
image

Versioner:
Hajk 3 Beta. Testad huvudsakligen i Microsoft Edge. Ej testad i mobiltelefon.
Notera
Denna "issue" är en av de "Buggar/synpunkter på ÖP-verktygen" i HAJK 3 Beta" beskrivna 30 aug 2019 av användare i Uddevalla kommun.

@victoraxbom
Copy link
Contributor

Kopplingen mellan kapitel och karta fungerar här hos oss! Länken till de relaterade kapitlen dyker upp både på de små "lagerkorten" och i lagerhanteraren.

@jacobwod
Copy link
Member

@victoraxbom Tacksam om test med senaste master kunde göras – kan hända att Breadcrumbs är borta så det vore bra om du kunde testa med dina oförändrade konfigurationsfiler som du vet visade Breadcrumbs i tidigare Hajkversioner.

@victoraxbom
Copy link
Contributor

@jacobwod Har testat mot master med konfigurationsfiler som fungerat tidigare men nu fungerar det inte längre, det kommer alltså inga Breadcrumbs när man trycker upp en karta från öp-fönstret. Inte heller kopplingen mellan kartlager och textkapitel verkar fungera som den ska. Kartan kraschar nu, det verkar inte som den hittar fönstret med öp-texten.

@jacobwod
Copy link
Member

Bra att veta, misstänkte att det kunde ha hänt nåt. Det är bara att ta tag i det, men jag har dessvärre inga konfigurationsfiler som jag med säkerhet vet fungerar (har inte använt funktionen).

Vet du om det var nåt speciellt man skulle slå på för att Breadcrumbs skulle komma? Jag har en enkel ÖP-fil som bara har en "Visa i karta" knapp, och när jag trycker på den så zoomar den mig till rätt läge och öppnar upp rätt lager. Däremot får inte heller jag några Breadcrumbs och därför funderar jag på om det var nåt mer man skulle göra eller om de kom automatiskt?

@victoraxbom
Copy link
Contributor

Jag kan skicka över de konfigurationsfiler som fungerade här hos oss. Lagerhanteraren behövde vara satt som en widget för att Breadcrumbs skulle synas. Lite dumt, det borde vara en separat inställning i admin om man vill använda dem eller inte, helt frikopplat från widget/toolbar.

@jacobwod
Copy link
Member

Just det, LayerSwitcher ska vara target=left|right. Men då vet jag att det var det som gjorde om de syns eller inte hos dig och testar. Tack för förslaget om filerna men när jag tänker på det så kommer det knappast fungera speciellt bra: er layers.json innehåller väl också en mängd referenser som står internt och jag kommer då ändå behöva göra om det mesta för att nåt ska visas.

@victoraxbom
Copy link
Contributor

Ja det stämmer, de flesta är interna. Så nog bättre du skapar en egen

@victoraxbom
Copy link
Contributor

Om du hinner när du ändå är inne i koden får du gärna kika på att lägga in en inställning i admin för dem.

@jacobwod
Copy link
Member

Är aldrig inne i mapservice faktiskt. Skulle kunna göra en flagga i new-client som läses men nån som sitter med .NET får gärna ta och fixa så flaggen faktiskt skrivs i mapservice.

jacobwod added a commit that referenced this issue Sep 16, 2019
- Breadcumbs show up only if LayerSwitcher is confiugred as Widget plugin. This may, or may not, be deserved. We should discuss it. An alternative is to add a configuration option that control Breadcrumbs visibility.
- This should do it for #279 and #276, please test and supply feedback.
@victoraxbom
Copy link
Contributor

victoraxbom commented Sep 18, 2019

Har testat senaste master och nu dyker tyvärr inte länkningen till kapitel upp alls, varken under lagret eller på Breadcrumbs. Men du kanske är inne och jobbar med de delarna @jacobwod ?

@victoraxbom
Copy link
Contributor

Det var configfilen som var korrupt, nu syns länkningen igen. Kraschar dock fortfarande när man försöker öppna kapitlet.

@jacobwod jacobwod added this to the 3.0 milestone Sep 18, 2019
@jacobwod
Copy link
Member

OK, kan du lämna nån skärmdump och markera i den var du klickar när kraschen inträffar? Får du Reacts felmeddelande så får du gärna klistra in en dump här också, så har jag lite mer att gå på. 👍

@victoraxbom
Copy link
Contributor

Det är när man klickar på dessa:

image

image

Felet:

image

Sen är det motsvarande fel då i Breadcrumb.js. Misstänker att det har att göra med hur du har byggt om hanteringen av fönstren. Den verkar inte hitta informative-fönstret längre

@jacobwod
Copy link
Member

Aja, det ska jag kunna lösa bara jag återskapar problemet här. Hur ställer du in för att få en sån knapp, kan du visa en dump från admin också?

@victoraxbom
Copy link
Contributor

Toppen! Knapparna dyker upp bara man har en kartlager kopplad till ett kapitel i översiktsplanstexten.

image

@jacobwod
Copy link
Member

@victoraxbom @ingvar-uddevalla Ska funka nu, återkoppla gärna om det är ok hos er med.

@victoraxbom
Copy link
Contributor

@jacobwod Snabbt fixat! Kopplingen fungerar nu, dock öppnas inte dokumentfönstret upp när man trycker som det gjorde tidigare, vilket gör att man inte tror att det händer något. Är ju även då kopplat till biten om att ha flera fönster öppna samtidigt.

@jacobwod
Copy link
Member

Helt rätt, funderat på det också men lät det vara i nuläget. Kanske bättre att ändå tvinga att Informative-fönstret öppnas automatiskt, så man ser att det händer saker. Ska kolla på att lägga in det.

@victoraxbom
Copy link
Contributor

Kan vara bra att det öppnas automatiskt om det inte redan är öppet. Om du har tid får du gärna kolla på det

@ingvar-uddevalla
Copy link
Contributor Author

Bra. Ska kolla detta imorgon. (Vi har ju en autoimatinstallation som körs varje natt). Jag har dock problem med att verifiera övriga buggar trots uppdaterade backend-lösningar m.m. TRor det ligger knepigheter i mina config-filer. T ex så går det fortfarande inte att spara Breadscrums-inställningarna m.m i oversiktsplan.json-filen i min miljö (se #276) och därmed kan jag inte testa detta heller.

@jacobwod jacobwod reopened this Oct 10, 2019
@jacobwod
Copy link
Member

Kan vara bra att det öppnas automatiskt om det inte redan är öppet. Om du har tid får du gärna kolla på det

Precis, det är det som var kvar och nu är det fixat i och med kommande commit. Hold on.

@victoraxbom
Copy link
Contributor

Kan vara bra att det öppnas automatiskt om det inte redan är öppet. Om du har tid får du gärna kolla på det

Precis, det är det som var kvar och nu är det fixat i och med kommande commit. Hold on.

Toppen! Jag har testat och det verkar fungera bra nu!

@jacobwod
Copy link
Member

@victoraxbom Jag har ifs ett problem som återstår: när man kör via npm start så fungerar allt perfekt. Men när jag bygger ut med npm run build så får jag följande fel när jag klickar på "pilen" i Breadcrumbs som då ska öppna Informativeverktyget:
image

Kan du bygga ut hos dig och se om det fungerar eller är lika illa? Hoppas att det är nåt lokalt cacheproblem bara, men så irriterande att inte bli av med det...

@jacobwod jacobwod reopened this Oct 11, 2019
@jacobwod
Copy link
Member

jacobwod commented Oct 11, 2019

OK, jag vet vad det är och är på väg att lösa det.

Har med webpacks minification att göra, den ändrar klassnamn etc, och vi har ett ställe i koden där window.type genereras automatiskt utifrån this.constructor.name. Se även webpack-contrib/uglifyjs-webpack-plugin#269.

@jacobwod
Copy link
Member

262a8f6 closes #279.

@jacobwod jacobwod self-assigned this Oct 11, 2019
@victoraxbom
Copy link
Contributor

@jacobwod Bra att det löste sig, Kan bekräfta att det fungerar bra att bygga och köra nu.

@jacobwod
Copy link
Member

Bra, tack för återkopplingen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants