search

TIM-JYU / TIM

TIM (The Interactive Material) is an open-source cloud-based platform for creating interactive learning documents.
https://tim.education/view/about/en-US
MIT License
13 stars 4 forks source link

Editorissa ei toimi Insert-tabin Slide- eikä Table-nappi #1042

Closed dezhidki closed 2 years ago

dezhidki commented 6 years ago

In GitLab by @Smibu on Jan 18, 2018, 08:54

dezhidki commented 6 years ago

In GitLab by @Smibu on Jan 18, 2018, 09:06

closed via commit 106917abebb814803a3bde7e28a83ee78099d962

dezhidki commented 6 years ago

In GitLab by @vesal on Jan 18, 2018, 10:52

Mikä tuossa oli vikana?

Vesa

dezhidki commented 6 years ago

In GitLab by @Smibu on Jan 18, 2018, 11:05

Templaatti väärin parissa kohtaa, ks. tuo yllä oleva commit.

Tuolta muuten saisi päälle sen, että commiteista tulee email. Pistänkö päälle?

dezhidki commented 6 years ago

In GitLab by @vesal on Jan 18, 2018, 11:13

Tuolta muuten saisi päälle sen, että commiteista tulee email. Pistänkö päälle?

Ehkä ei kannata. Niitä vo i tulla aika paljon. Paremmpi voisi olla liittää joku pikkuviesti ticketin sulkemiseen viimeistään että mitä periaatteessa muutettiin ja/tai linkki siihen committiin.

Tosin se committi linkki taitaa olla tuossa edellisessä viestissä jo, mutta en sitä pinellä nähnyt.

Muutta esim tässä tapauksessa se kiinostava tietysti että mitä oli muuttunut että nuo olivat väärään paikkaan, ihan vaan jotta jos vastaavia itse muuttelee, niin tietää että niissäkinp itää tuo ottaa huomioon. Mää esim kun ehtisin, haluaisin tehdä sen, että omia tabeja voi tehdä muuallekin kuin plugins alle.

Vesa

dezhidki commented 6 years ago

In GitLab by @Smibu on Jan 18, 2018, 11:40

Ehkä ei kannata. Niitä vo i tulla aika paljon.

Tai oikeastaan jokaisesta puskusta tulisi. Jos on monta committia samassa puskussa, niin ne kai olisivat samassa mailissa.

Tosin se committi linkki taitaa olla tuossa edellisessä viestissä jo, mutta en sitä pinellä nähnyt.

Joo, ja tähän korttiin ilmestyy linkki korjaavaan committiin (kuten nyt), jos muistan laittaa commit-viestiin, että fixes #xxx, missä xxx on kortin numero.

Muutta esim tässä tapauksessa se kiinostava tietysti että mitä oli muuttunut että nuo olivat väärään paikkaan, ihan vaan jotta jos vastaavia itse muuttelee, niin tietää että niissäkinp itää tuo ottaa huomioon.

Tää vika oli jäänyt siitä refaktoroinnista. Osan metodeista siirsin apuluokkaan (siis juuri ne, joita aiemmin vaihdeltiin niiden setAceControllerFunctions ja setTextAreaControllerFunctions-metodien avulla). Ja tosiaan parissa kohtaa templaatteja kutsuttiin siksi väärää oliota. TypeScripthän ei noita Angular1-templaatteja osaa katsoa, että onko niissä vikaa.

Mää esim kun ehtisin, haluaisin tehdä sen, että omia tabeja voi tehdä muuallekin kuin plugins alle.

Joo. Tuota kun tekee, niin olisi hyvä saada niin, ettei missään tarvitsisi kutsua $compilea itse. Noiden tabien kanssa osa työstä tehdään suotta jQueryllä ja sen takia tuota $compilea joudutaan käyttämään.

dezhidki commented 6 years ago

In GitLab by @Smibu on Jan 22, 2018, 10:04

mentioned in commit 106917abebb814803a3bde7e28a83ee78099d962