phoenix icon indicating copy to clipboard operation
phoenix copied to clipboard

Make collections info panel compact

Open 9inpachi opened this issue 4 years ago • 23 comments

The idea is to make the Collections table have a different layout for mobile / smaller views.

This is the current view of the collections info panel.

image

On smaller screens (and small panel size), the data gets cut off and we have to scroll horizontally. It would be great if we could have a different layout on smaller view. (Maybe something where each column is on a separate line)

9inpachi avatar Jan 20 '21 18:01 9inpachi

Seems like a good issue to start with. Looking into this. Thanks

ksraj123 avatar Jan 20 '21 18:01 ksraj123

@9inpachi Can I work on this issue ?

karthikshetty03 avatar Feb 16 '21 07:02 karthikshetty03

Yes sure. Just give it a try and create a pull request if it works.

9inpachi avatar Feb 16 '21 09:02 9inpachi

@9inpachi
Can you please give me the link to the location of the code or give me some more information regarding this issue? Also, can you tell me where can I find this page in the online version?

karthikshetty03 avatar Feb 25 '21 19:02 karthikshetty03

You can find it if you go to an experiment from the main page (for example, https://hepsoftwarefoundation.org/phoenix/#/atlas) and click on the "i" icon in the bottom UI menu to open the collections info panel.

Also please take a look at guides/users.md to understand how everything works.

If this information is not enough to know the code location then let me know. :)

On Fri, Feb 26, 2021, 12:33 AM Karthik Shetty [email protected] wrote:

@9inpachi https://github.com/9inpachi Can you please give me the link to the location of the code or give me some more information regarding this issue? Also, can you tell me where can I find this page in the online version https://hepsoftwarefoundation.org/phoenix/?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/HSF/phoenix/issues/206#issuecomment-786148676, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIZVY6JACUJWYW7KLR5H6WLTA2QXFANCNFSM4WLCNQSA .

9inpachi avatar Feb 25 '21 20:02 9inpachi

Yes, I got the code location and will try implementing the functionality. Thank You!

karthikshetty03 avatar Feb 26 '21 04:02 karthikshetty03

According to your suggestion, the UI on small screens should be such that the columns are one below the other, right?

karthikshetty03 avatar Feb 26 '21 06:02 karthikshetty03

Indeed.

On Fri, Feb 26, 2021, 11:10 AM Karthik Shetty [email protected] wrote:

According to your suggestion, the UI on small screens should be such that the columns are one below the other, right?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/HSF/phoenix/issues/206#issuecomment-786435946, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIZVY6L2PCEDVGMXDYHVD7LTA43NPANCNFSM4WLCNQSA .

9inpachi avatar Feb 26 '21 06:02 9inpachi

Screenshot from 2021-02-26 15-59-18 @9inpachi Is this fine?

karthikshetty03 avatar Feb 26 '21 10:02 karthikshetty03

@karthikshetty03 Good progress so far! Thanks.

The next step would be to show the column name for each value at the left. Something like.

columnName1: The value columnName2: Some other value

Also it'd be interesting if it worked with resizing the overlay on desktops as well.

9inpachi avatar Feb 26 '21 11:02 9inpachi

The next step would be to show the column name for each value at the left. Something like.

columnName1: The value columnName2: Some other value

Yeah, will do that

karthikshetty03 avatar Feb 26 '21 12:02 karthikshetty03

Also it'd be interesting if it worked with resizing the overlay on desktops as well.

@9inpachi As in if the table requires a horizontal scroll on the desktop, then I have to display all columns one below the other as well?

karthikshetty03 avatar Feb 26 '21 12:02 karthikshetty03

Yes exactly.

On Fri, Feb 26, 2021, 5:22 PM Karthik Shetty [email protected] wrote:

Also it'd be interesting if it worked with resizing the overlay on desktops as well.

@9inpachi https://github.com/9inpachi As in if the table requires a horizontal scroll on the desktop, then I have to display all columns one below the other as well?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/HSF/phoenix/issues/206#issuecomment-786615637, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIZVY6MV5I5EB7V7C53VIDLTA6HBNANCNFSM4WLCNQSA .

9inpachi avatar Feb 26 '21 12:02 9inpachi

@9inpachi Apologies for the delay, I was busy with my college exams for a week. I resumed working on this but am stuck on the part on how to determine whether there is a horizontal scrollbar or not. Like if there is a scrollbar, then the layout will be changed to a small screen view (each column on a separate line) I tried looking around the overlay-component.ts file, where the overlay window resizing is handled but the clientWidth and scrollWidth of the overlayCard do not seem to be of much help. Any hints or suggestions on how to go about this?

karthikshetty03 avatar Mar 09 '21 19:03 karthikshetty03

Not a problem.

Could you open up a pull request so I can take a look (probably tomorrow) at your approach so far? I can comment better after that.

On Wed, Mar 10, 2021, 12:42 AM Karthik Shetty [email protected] wrote:

@9inpachi https://github.com/9inpachi Apologies for the delay, I was busy with my college exams for a week. I resumed working on this but am stuck on the part on how to determine whether there is a horizontal scrollbar or not. Like if there is a scrollbar, then the layout will be changed to a small screen view (each column on a separate line) I tried looking around the overlay-component.ts file, where the overlay window resizing is handled but the clientWidth and scrollWidth of the overlayCard do not seem to be of much help. Any hints or suggestions on how to go about this?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/HSF/phoenix/issues/206#issuecomment-794359721, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIZVY6IIL7TECQC6IKLHE7DTCZ2YPANCNFSM4WLCNQSA .

9inpachi avatar Mar 09 '21 19:03 9inpachi

I used media queries that changes the layout for small screens.

Could you open up a pull request so I can take a look (probably tomorrow) at your approach so far?

Yeah, will do that.

karthikshetty03 avatar Mar 10 '21 03:03 karthikshetty03

Screenshot from 2021-03-10 12-35-11

@9inpachi this works fine for mobile/smaller views. Need to work on resizing the overlay on desktops. I have opened a PR: #228

karthikshetty03 avatar Mar 10 '21 08:03 karthikshetty03

Yes, I got the code location and will try implementing the functionality. Thank You!

can you share the code location please

AyushABD-17 avatar Mar 06 '23 09:03 AyushABD-17

hello can I work on this?

staru09 avatar Dec 11 '23 18:12 staru09

hello can I work on this?

Absolutely! Feel free to ask us with any questions.

EdwardMoyse avatar Dec 11 '23 23:12 EdwardMoyse

is it open to work???

ishqDehlvi avatar Dec 30 '23 14:12 ishqDehlvi

Not sure. @staru09 are you taking this? Please look at the earlier PR too.

EdwardMoyse avatar Dec 30 '23 22:12 EdwardMoyse

Is this open to work on?

StoneCoder123 avatar Feb 11 '24 19:02 StoneCoder123

This is still open.

EdwardMoyse avatar Dec 03 '24 10:12 EdwardMoyse

Hello, can I start working on this?

Pratz2005 avatar Dec 12 '24 09:12 Pratz2005

Yes, please do!

On Thu, 12 Dec 2024 at 10:51, Pratham @.***> wrote:

Hello, can I start working on this?

— Reply to this email directly, view it on GitHub https://github.com/HSF/phoenix/issues/206#issuecomment-2538401113, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABTTQSOB75J73YVFSDGE5VL2FFMDDAVCNFSM6AAAAABS5PQVRGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZYGQYDCMJRGM . You are receiving this because you commented.Message ID: @.***>

EdwardMoyse avatar Dec 12 '24 09:12 EdwardMoyse

Screenshot 2024-12-12 233714

does this look fine?

Pratz2005 avatar Dec 12 '24 15:12 Pratz2005

It does - thanks!

EdwardMoyse avatar Dec 12 '24 16:12 EdwardMoyse

ok ill make the PR

Pratz2005 avatar Dec 12 '24 16:12 Pratz2005

hey How to get started or contribute is there anything i can do

FARDEEN-785 avatar Jan 19 '25 10:01 FARDEEN-785