Github user michellephung commented on the pull request:

    https://github.com/apache/couchdb-fauxton/pull/543#issuecomment-144833299
  
    :( this toggle bar looks wierd here:
    ![screen shot 2015-10-01 at 8 10 58 
pm](https://cloud.githubusercontent.com/assets/836039/10230910/19215f50-6880-11e5-9b92-ae9f2927bdfd.png)
    
    lightening the border makes it look *less* weird, 
    ![screen shot 2015-10-01 at 8 11 03 
pm](https://cloud.githubusercontent.com/assets/836039/10230933/395b2652-6880-11e5-99d8-66f152a277ae.png)
    ![screen shot 2015-10-01 at 8 33 56 
pm](https://cloud.githubusercontent.com/assets/836039/10230935/3f21d9e6-6880-11e5-9e63-dbdfd94951ef.png)
    and then taking out the background, is an improvement still:
    ![screen shot 2015-10-01 at 9 13 42 
pm](https://cloud.githubusercontent.com/assets/836039/10231233/f0f054bc-6881-11e5-929e-b613056d6fab.png)
    
    but really, the style of the toggle still doesn't fit in with the overall 
design of the page.
    
    I do really like the search bar coming down. :+1:  Its okay that it 
overlaps the cards, it's pretty unintrusive. It is definitely an improvement on 
what was happening before, when it would just become unreachable at smaller 
widths. Lets keep this as a feature, this is not a bug to me. :D
    
    The table is okay. Not super happy about it, but if we stick with it, we'll 
really have to address the include_docs=true situation, because the default 
currently, is really confusing:
    ![screen shot 2015-10-01 at 8 29 19 
pm](https://cloud.githubusercontent.com/assets/836039/10231021/b1c1bc46-6880-11e5-9433-d40ab5db7144.png)
    
    It's all scary random strings, and if even if those _ids were legible, they 
wouldn't really give that much information, unless include_docs=true.
    
    I say take default  include_docs=true, AFTER we think about something to do 
about the databases which have way too much to show on the cards. Maybe 
throttle/truncate the data for those folks, by setting a max length for the 
cards? (just a first idea, better ideas welcome)
    
    
    there was some discussion before about moving the query things into to 
upper space of the rightmost section where the cards are, which is a good idea.
    
    Lets see what others have to say. 
    You've got me thinking about it though, more ideas will probably formulate 
in the days to come.
    
    :D looks cool though, just a little needs some polishing.
    
    In the meantime.... 
    1. I'd really love to see futon-style table inside the json cards:
    ![screen shot 2015-09-28 at 4 29 23 
am](https://cloud.githubusercontent.com/assets/836039/10231535/9d94cb3e-6883-11e5-9b09-da7b9a675bb2.png)
    ![screen shot 2015-10-01 at 9 40 23 
pm](https://cloud.githubusercontent.com/assets/836039/10231893/a76ebc3a-6885-11e5-841a-8f5e669683c2.png)
    (stick with existing font-family/size/color though. I just didnt have time 
to change the font here^^)
    
    sean/andrea mocked up something like this from before.  It's around 
somewhere. I remember sean showing it to us and i really like it. I didn't 
spend too much time on the colors, pick colors that match what's going already, 
or whatever, it'll be easy to change the colors in css. It's just the 
formatting of the table I'm after. This is hard to read: ![screen shot 
2015-09-28 at 4 29 08 
am](https://cloud.githubusercontent.com/assets/836039/10231537/a4ba5f6e-6883-11e5-80eb-498ef3c39f91.png).
 
    
    2. and the repeated thrice _id/_rev!!! to be taken out of the cards because 
they are taking up tons of space!
    ![screen shot 2015-10-01 at 9 51 25 
pm](https://cloud.githubusercontent.com/assets/836039/10232057/9e1000e4-6886-11e5-80a6-301c605c10d0.png)
    (if you have time :sweat_smile: )


---
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