cannin / ihop-reach

A web application to access biological data extracted from biomedical literature.
https://reach.nrnb-docker.ucsd.edu
GNU Lesser General Public License v3.0
4 stars 4 forks source link

Add Journal Title to Details Column to Site #54

Closed cannin closed 4 years ago

cannin commented 5 years ago

Add a column to the right of the sentence that has the Journal Title you have extracted.

RohitChattopadhyay commented 5 years ago

Desktop View image Mobile View image Requesting feedback Thanks

cannin commented 5 years ago

This is good I would move the date to the rightmost. And there needs to be more searches for the dates and journal.

On Tue, Jul 9, 2019 at 6:06 PM Rohit Rajat Chattopadhyay < notifications@github.com> wrote:

Desktop View [image: image] https://user-images.githubusercontent.com/19877300/60926257-9b56f300-a2c3-11e9-94ee-313085475d22.png Mobile View [image: image] https://user-images.githubusercontent.com/19877300/60926342-ce00eb80-a2c3-11e9-9fb9-62458f2b69f2.png Requesting feedback Thanks

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cannin/ihop-reach/issues/54?email_source=notifications&email_token=AAGA6DCUIZ3PSBCYSDNQ6FTP6UDVLA5CNFSM4H7AFOE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZRVUDA#issuecomment-509827596, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGA6DEJKTZPNPASDROTXB3P6UDVLANCNFSM4H7AFOEQ .

RohitChattopadhyay commented 5 years ago

Should I make it italic so that it is easy to distinguish? image

RohitChattopadhyay commented 5 years ago

This is good I would move the date to the rightmost. And there needs to be more searches for the dates and journal.

I have merged sentences and journal_title into the single search field, is that ok? Should I add Year as well in this field or separate for each? The placeholder message can be changed accordingly I think it will become clumsy if we add so many fields.

cannin commented 5 years ago

It would be best to leave these separate (year, sentence, journal) so they can be searched separately. I agree that too many columns are bad, but we haven't arrived at what the "right" columns are, so don't worry too much about this for now.

On Tue, Jul 9, 2019 at 6:33 PM Rohit Rajat Chattopadhyay < notifications@github.com> wrote:

This is good I would move the date to the rightmost. And there needs to be more searches for the dates and journal.

I have merged sentences and journal_title into the single search field, is that ok? Should I add Year as well in this field or separate for each? The placeholder message can be changed accordingly I think it will become clumsy if we add so many fields.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cannin/ihop-reach/issues/54?email_source=notifications&email_token=AAGA6DAKLCZEYDIL6QVFLMDP6UG2DA5CNFSM4H7AFOE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZRXKIY#issuecomment-509834531, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGA6DHBA2RZKD4OY2TUHLLP6UG2DANCNFSM4H7AFOEQ .

cannin commented 5 years ago

https://digital.gov/2014/10/28/trends-on-tuesday-8-ways-to-format-tables-for-responsive-web-design/

Options 2, 5, 1 would be my suggestions for mobile view (most to least preferred). If 5 then I would keep: PMC link, hypothesis_information, negative_info, organism, sentence.

RohitChattopadhyay commented 5 years ago

Options 2, 5, 1 would be my suggestions for mobile view (most to least preferred). If 5 then I would keep: PMC link, hypothesis_information, negative_info, organism, sentence.

Taking inspiration from option 2, I have tried the following layout for the small screen device. image This can be tested here Desktop design is untouched, requesting feedback Thanks

RohitChattopadhyay commented 5 years ago

Option 2 has been deployed, requesting feedback Link: https://reach.nrnb-docker.ucsd.edu/details/chebi:CHEBI:1

The table filters have been improved to support regex