Github user benkeen commented on the pull request:

    https://github.com/apache/couchdb-fauxton/pull/145#issuecomment-62922729
  
    Thanks @robertkowalski!
    
    Yeah, I do see your point... I just wish it could be made explicit that the 
`toggleTrayBtnSelector` had special meaning to the parent View so you wouldn't 
have to look up the Components.Tray code to understand it. I tend to favour 
landing on the side of code clarity, even if does mean a little more code. 
    
    But point taken. Since that'll require a (small) refactor of all the trays, 
I'll create a follow-up ticket to this one that cleans it up across the board. 
:)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to