cozy / cozy-emails

Email Client for Cozy
GNU Affero General Public License v3.0
66 stars 41 forks source link

Loosing focus in message list when remove a message (Suppr) #759

Closed misstick closed 8 years ago

misstick commented 8 years ago

Actually _fixCurrentMessage should do that stuff but it doesnt work.

When a message is deleted :

When a user navigate with keyboard into messageList :

frankrousseau commented 8 years ago

Could you add comments telling what your function does and why you choose to do it that way (if needed)?

misstick commented 8 years ago

@frankrousseau : I thought function names were the most explicit here, didnt it?

frankrousseau commented 8 years ago

Yep, but you can explain how the function works and why you choosed to do it that way.

frankrousseau commented 8 years ago

It's ok for me. @aenario Do you think we can merge it?

misstick commented 8 years ago

@frankrousseau : I'm adding comments on store, to be more explicit as you asked before