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

Update grido.js #265

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update grido.js #265

wants to merge 1 commit into from

Conversation

djdaca
Copy link
Contributor

@djdaca djdaca commented Feb 26, 2016

Nevím jak to udělat lépe, ale podmínka td:not(.actions a) určitě nefunguje, chybu jsem vyřešil odstraněním odkazu ze selectoru což způsobí že nelze klikat v celé oblasti akcí.

Je otázkou jestli je to potřeba.

Nevím jak to udělat lépe, ale podmínka td:not(.actions a) určitě nefunguje, chybu jsem vyřešil odstraněním odkazu ze selectoru což způsobí že nelze klikat v celé oblasti akcí. 

Je otázkou jestli je to potřeba.
@o5
Copy link
Owner

o5 commented Feb 26, 2016

Ahoj, zkoušel jsem to a máš pravdu, že to teď nefunguje. Ale:

  1. Chtělo by to opravit se zachováním původního záměru (tedy kliknutí v buňce s akcema - ale zároveň mimo <a> - způsobí stejné jako kliknutí na checkbox vlevo)
  2. Pull requestu chybí vygenerované assets/dist scripty (koukni na CONTRIBUTING -> Working with assets

@djdaca
Copy link
Contributor Author

djdaca commented Feb 27, 2016

Bylo to spíše ohlášení chyby a možný quick fix, přiznam se že o původním záměru jsem přemýšlel ale behem urcité doby mě žádná náprava chyby nenapadla, snad v budoucnu

@o5 o5 added the CLIENT-SIDE label May 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants