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

Chyba v řazení atributů při načtení mineru #95

Open
vojir opened this issue Jul 12, 2016 · 3 comments
Open

Chyba v řazení atributů při načtení mineru #95

vojir opened this issue Jul 12, 2016 · 3 comments

Comments

@vojir
Copy link
Member

vojir commented Jul 12, 2016

Při načtení mineru nedojde k seřazení atributů - jsou seřazeny až při přidání některého z atributů do vzoru pravidla.

attributes - error

@PreVaDu
Copy link
Contributor

PreVaDu commented Jul 13, 2016

Opraveno. Jen to teď nevypadá tak hezky, protože proběhne řazení i s animací po načtení, takže se to chová, jakoby to dělalo něco, co uživatel neovlivní.
Je tato chyba nová, nebo jsme si ji dříve nevšimli? Abych to případně zkusil pořešit na úrovni načítání dat.

@vojir
Copy link
Member Author

vojir commented Jul 15, 2016

Asi jsme si toho dřív jen nevšimli. Předpokládám správně, že při výchozím správném seřazení položek k žádné animaci nedojde? (můžu doplnit seřazení seznamu atributů do PHP připravujícího pole s daty)

@PreVaDu
Copy link
Contributor

PreVaDu commented Jul 16, 2016

Ano, pokud jsou načteny atributy rovnou seřazené, funkce sice proběhně, ale animace není třeba.

Všiml jsem si ovšem jiné chyby - když skryji atribut a znovu načtu miner, zobrazí se sice správně "Show hidden", ale skrytý atribut je stále viditelný... Pamatuji si, jak jsme ty skryté řešili a myslím, že to bylo v pořádku. Nemohlo se tam celkově něco změnit na logice, co ovlivnilo tyto funkcionality? (řazení, skrývání apod.) Jestli se nezměnil třeba backend, že již nevrací seřazené atributy, nevrací zvlášť hidden apod.

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

2 participants