New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Journal Title to Details Column to Site #54
Comments
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 < ***@***.***> 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
<#54?email_source=notifications&email_token=AAGA6DCUIZ3PSBCYSDNQ6FTP6UDVLA5CNFSM4H7AFOE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZRVUDA#issuecomment-509827596>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGA6DEJKTZPNPASDROTXB3P6UDVLANCNFSM4H7AFOEQ>
.
|
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 |
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 < ***@***.***> 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
<#54?email_source=notifications&email_token=AAGA6DAKLCZEYDIL6QVFLMDP6UG2DA5CNFSM4H7AFOE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZRXKIY#issuecomment-509834531>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGA6DHBA2RZKD4OY2TUHLLP6UG2DANCNFSM4H7AFOEQ>
.
|
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. |
Taking inspiration from option 2, I have tried the following layout for the small screen device. |
Option 2 has been deployed, requesting feedback The table filters have been improved to support regex |
Add a column to the right of the sentence that has the Journal Title you have extracted.
The text was updated successfully, but these errors were encountered: