TODO - Githubissues

korsvanloon commented 9 years ago

on page load:

get all the ExplanationRequests and Explanations of this page (url)
find parent Elements and replace them with the Explanation(Request)s

on text selection right click:

filter out text selections outside the article
get parent Element (e.g. <p>)
send word, url, and parent Element to server

maartensteinfort commented 9 years ago

Heb het net even bekeken, en zag het knopje al staan van het context menu, ziet er goed uit!

Wat bedoel je met 'parent elements'?

korsvanloon commented 9 years ago

De paragraaf (oid) waar de selected text in staat. Het probleem is nu dat je uit zo'n click event (van de contextmenu) alleen de selectedtext en de url kunt halen, dus moeten we nog wat zin hoe we de paragraaf mee kunnen sturen waar de selectedtext in staat.

korsvanloon commented 9 years ago

Fixed text selection right click

korsvanloon commented 9 years ago

Als je nu met je rechtermuisknop op een selectie een explanation request doet, dan verschijnt er een linkje.

Verder heb ik een opzetje gemaakt voor de popup die vanuit de js een html lijstje uitprint. Dit is handig voor later, want dan moeten we een lijstje opbouwen vanuit een HttpRequest naar de server.

maartensteinfort commented 9 years ago

Goeie. Ik ga er dit weekend ook even aan zitten. Even kijken hoe je dit allemaal hebt gedaan!

maartensteinfort commented 9 years ago

Goedesmorgens!

Ik heb alles even beken en zie inderdaad het linkje. Ik heb een paar (amateur) vragen die graag zou stellen, omdat het toch allemaal buiten mijn kennis gaat.

Daarom weekend heb ik even een 10 uur codecademy javascript 'beginners' course genomen.

maartensteinfort commented 9 years ago

Ik krijg trouwens niet een script uit popup.js aan de praat in popup.html.

korsvanloon commented 9 years ago

We kunnen na IR er wel even aan zitten

2015-03-16 9:34 GMT+01:00 Maarten Steinfort notifications@github.com:

Ik krijg trouwens niet een script uit popup.js aan de praat in popup.html.

— Reply to this email directly or view it on GitHub https://github.com/korsvanloon/skiir_plugin/issues/1#issuecomment-81503314 .

maartensteinfort commented 9 years ago

ja goed plan.

Op 16 mrt. 2015, om 10:13 heeft korsvanloon notifications@github.com het volgende geschreven:

We kunnen na IR er wel even aan zitten

2015-03-16 9:34 GMT+01:00 Maarten Steinfort notifications@github.com:

Ik krijg trouwens niet een script uit popup.js aan de praat in popup.html.

— Reply to this email directly or view it on GitHub https://github.com/korsvanloon/skiir_plugin/issues/1#issuecomment-81503314 .

— Reply to this email directly or view it on GitHub https://github.com/korsvanloon/skiir_plugin/issues/1#issuecomment-81520922.

maartensteinfort commented 9 years ago

Jojo

Ik wil even werken aan de dialog.html en dialog.js maar ik zie nu dat als ik de meest recente versie binnenhaal, dat ik nog dingen moet comitten...

Maar ik heb niet echt iets toegevoegd. Weet jij hoe ik nu de master copy als basis kan gebruiken? (zonder dat ik nu mijn changes push)

schermafbeelding 2015-03-19 om 13 30 24

korsvanloon commented 9 years ago

op de console kun je zien welke bestanden je hebt aangepast met: "git status" die bestanden moet je dan resetten: "git checkout -- path/to/file"

let op de spaties

2015-03-19 13:30 GMT+01:00 Maarten Steinfort notifications@github.com:

Jojo

Ik wil even werken aan de dialog.html en dialog.js maar ik zie nu dat als ik de meest recente versie binnenhaal, dat ik nog dingen moet comitten...

Maar ik heb niet echt iets toegevoegd. Weet jij hoe ik nu de master copy als basis kan gebruiken? (zonder dat ik nu mijn changes push)

[image: schermafbeelding 2015-03-19 om 13 30 24] https://cloud.githubusercontent.com/assets/10925490/6730361/232c633e-ce3c-11e4-8cb9-38c7930f1ec7.png

— Reply to this email directly or view it on GitHub https://github.com/korsvanloon/skiir_plugin/issues/1#issuecomment-83536908 .

korsvanloon commented 9 years ago

Ik heb nu op die ene pagina 1 explanation request (exReq) met een rode underline en 1 explanation (ex) met een groene underline staan. Als je over de ex hovert dan zie je de uitleg.

Je kunt nu:

Een exReq aanmaken met je rechtermuisknop
Een exReq vervullen. Er verschijnt dan een dialoogje en als je die vult, dan verandert de exReq in een ex.

Ik doe nog niets met die popup daar boven.

maartensteinfort commented 9 years ago

Top! Goeie!

Ik zag al inderdaad dat je bezig was. Heb hem nu geopend, maar ik zie nu alleen als ik de extensie probeer te gebruiken bij chrome dat:

Laden van extensie mislukt vanuit: ~/Github/skiir_plugin
Kan JavaScript 'angular.min.js' niet laden voor het inhoudsscript.

Ik wil nu bezig gaan met de dialoog die popupt, maar die dialog.html file is weer weg zie ik?

korsvanloon commented 9 years ago

Moet je nog een keer updaten. Ik had als eerst angular bestanden toegevoegd (en geregistreerd in de manifest.json), maar die heb ik toen weer weggehaald. De dialog.html heb ik iid ook weer weggegooid. Ik plak nu een <dialog id="skiir-dialog"> element in de body.

2015-03-20 14:05 GMT+01:00 Maarten Steinfort notifications@github.com:

Top! Goeie!

Ik zag al inderdaad dat je bezig was. Heb hem nu geopend, maar ik zie nu alleen als ik de extensie probeer te gebruiken bij chrome dat:

Laden van extensie mislukt vanuit: ~/Github/skiir_plugin Kan JavaScript 'angular.min.js' niet laden voor het inhoudsscript.

Ik wil nu bezig gaan met de dialoog die popupt, maar die dialog.html file is weer weg zie ik?

— Reply to this email directly or view it on GitHub https://github.com/korsvanloon/skiir_plugin/issues/1#issuecomment-84007427 .

maartensteinfort commented 9 years ago

Ah oke, heb nu

       //"ui-bootstrap-tpls-0.12.1.min.js",

weggehaald en dan doet ie het! Ziet er heel goed uit. Ik probeerde je net even te bellen, want wil even weten waar ik nu aan kan werken

korsvanloon / skiir_plugin

TODO #1