KIZI / EasyMiner-MiningUI

Web UI for mining in new version EasyMiner Project
Apache License 2.0
3 stars 1 forks source link

Doladění funkčnosti Rule Clipboard a Knowledge Base #66

Closed vojir closed 9 years ago

vojir commented 9 years ago
vojir commented 9 years ago

K chybě s layoutem dojde v případě, kdy je zobrazena knowledge base, je zobrazen rule set s větším množstvím pravidel a přes změnu rule setu je vyměněn za rule set s menším počtem pravidel. Chyba se projevuje v posledních verzích Firefoxu i Chrome.

PreVaDu commented 9 years ago

Setrvává chyba stále? Poslední verze FF i Chrome a nic takového nepozoruji. Může to souviset se scrollováním na původní místo, ale u mě se to chová korektně. Měním z rulesetu s 19 pravidly (stránkování, ale dal jsem si i výpis všech) na ruleset s 1.

vojir commented 9 years ago

Chyba přetrvává stále, testováno ve FF 40.0.2 na win 10. Zároveň po prvním projevu chyby se chyba objevuje i při stránkování.

Zároveň mi aktuálně nefunguje označení aktuální aktivní stránky rule clipboard při větším množství stran (aktuálně 3). I když se načte další strana pravidel, tak je označená vždy 1.

PreVaDu commented 9 years ago

Tak konečně nasimulováno - jedná se o případ, kdy je overlay se změnou rulesetu vyšší než viditelná oblast, v takovém případě dojde k rozhození - závislost na velikosti rulesetu je spíše malá, jde pouze o přechod z velkého na malý. Chyba byla v overlay - zůstávala jeho výška i po skrytí a tak se natahoval main díky section overlay víc, než body a footer tedy nepasoval dolu. Opraveno.

Označení aktuální stránky nefunguje v KB, v rule clipboard tuto chybu nepozoruji. V KB nejde kvůli tomu, že z back-endu nedostávám s JSON s rulesCount (dříve jsem dostával, u MR tuto informaci dostávám).

vojir commented 9 years ago

Ještě prosím to označení aktuální stránky v KB. Počet pravidel v daném rulesetu je vracen v každém seznamu pravidel:

{"ruleset":{"id":5,"name":"Stanislav Vojíř","description":"","rulesCount":21},"rules":{"1487054":{"id"
:1487054,"text":"language(English_language) & country(United_States) & mediaType(Paperback) → rating(0)","a":3658,"b":2533,"c":35524,"d":30656,"selected":"0"}}}
PreVaDu commented 9 years ago

tak nakonec chyba jinde... děkuji a opraveno :)