search

stephenmcd / grappelli-safe

Admin skin for Mezzanine
Other
25 stars 59 forks source link

remove inappropriate try to focus first input field #71

Closed gradel closed 8 years ago

gradel commented 8 years ago

constantly throwing Uncaught TypeError: Cannot read property 'focus' of null. Not looked around much, but it seems that the first input field gets the focus anyway.

stephenmcd commented 8 years ago

Seems like overkill. What kind of field is causing the error? Why not just wrap in a try/catch?

gradel commented 8 years ago

adminform seems to lack the attribute first_field and the focus part is already handled in line 77 in the way django itself does it now.

stephenmcd commented 8 years ago

That makes sense, thanks a lot.