layer5 icon indicating copy to clipboard operation
layer5 copied to clipboard

Partners name gets cropped on bigger screen

Open Shubhcs01 opened this issue 3 years ago β€’ 8 comments
trafficstars

Current Behavior

RedHat and Vmware's name gets cropped on the bigger screen.

Screenshot 2022-08-31 203952

Desired Behavior

Screenshot 2022-08-31 212616


Contributor Resources and Handbook

The layer5.io website uses Gatsby, React, and GitHub Pages. Site content is found under the master branch.

Shubhcs01 avatar Aug 31 '22 16:08 Shubhcs01

Can I work on this issue?

adarsh500 avatar Aug 31 '22 16:08 adarsh500

Thank you for filing this issue, however, this is actually by design as we're not trying to list all partners, users, customers and so the effect is just that of cutting off the list at some point. We need to add more logos in order to have that sense conveyed in the two row view. That's what we should focus on in this issue. To start, let's add Docker's logo and F5's logo.

leecalcote avatar Aug 31 '22 16:08 leecalcote

Thank you for filing this issue, however, this is actually by design as we're not trying to list all partners, users, customers and so the effect is just that of cutting off the list at some point. We need to add more logos in order to have that sense conveyed in the two row view. That's what we should focus on in this issue. To start, let's add Docker's logo and F5's logo.

Okay, I get it. So, just to clarify I have to add Docker and F5's logo in the partner section right?

Shubhcs01 avatar Aug 31 '22 17:08 Shubhcs01

Hey @Shubhcs01 ! I am new to the community and want to work on this issue. Can you please assign it to me?

tinniaru3005 avatar Sep 08 '22 21:09 tinniaru3005

Thanks for raising your hand, @tinniaru3005. πŸ‘ Please note that the requirement has shifted.

leecalcote avatar Sep 08 '22 21:09 leecalcote

Thanks for raising your hand, @tinniaru3005. πŸ‘ Please note that the requirement has shifted.

Hello @leecalcote . Thanks for assigning the issue to me. Can you please explain to me the current requirements? I also wanted to confirm if the changes are required on this page. layer5

tinniaru3005 avatar Sep 08 '22 22:09 tinniaru3005

@tinniaru3005, not quite. Thanks for jumping in. The page that you’re referring to is the project page. The page in question is the homepage. Two new logos are requested. They must be SVGs and grayscaled.

leecalcote avatar Sep 09 '22 03:09 leecalcote

Hey @Shubhcs01 ! I am new to the community and want to work on this issue. Can you please assign it to me?

Hi @tinniaru3005 , first of all welcome to the community 🀝. I am slight busy in some college works so yeah please go on and solve the issue. Also if you need any help just drop a comment. Thanks @leecalcote for explaining the issue.πŸ™‚

Shubhcs01 avatar Sep 09 '22 18:09 Shubhcs01

Clearing assignees to allow others for taking it.

Nikhil-Ladha avatar Sep 30 '22 05:09 Nikhil-Ladha

Hello @Nikhil-Ladha and @leecalcote, If no one is working on the Issue can You assign the issue to me? I m new to the community

SaiRev0 avatar Oct 01 '22 12:10 SaiRev0

Yes, @SaiRev0 you can work on this. Though, the requirement is slightly different as mentioned in this comment

Nikhil-Ladha avatar Oct 01 '22 13:10 Nikhil-Ladha

@Nikhil-Ladha can you preferably update the issue title and desc as per this comment https://github.com/layer5io/layer5/issues/3133#issuecomment-1233191687 so that there's no confusion.

(I am not able to edit it)

kamalbuilds avatar Oct 03 '22 17:10 kamalbuilds

@Nikhil-Ladha So I have to add 1 more row for the logos of the partners??

am I correct with that??

SaiRev0 avatar Oct 08 '22 09:10 SaiRev0

Yes, and probably move a couple of them in the above row, to the new one.

Nikhil-Ladha avatar Oct 08 '22 10:10 Nikhil-Ladha

Yes, and probably move a couple of them in the above row, to the new one.

Hello @Nikhil-Ladha, I have a suggestion. Can he add a sliding effect instead of diving it between two rows? This will look good, I think. If I can contribute, then please let me know.

TiwariAbhishek23 avatar Oct 08 '22 14:10 TiwariAbhishek23

Nope, the overflow is by design and intentional

Nikhil-Ladha avatar Oct 08 '22 14:10 Nikhil-Ladha

Nope, the overflow is by design and intentional

Ok, thanks. But if the issue exists, then my setup is ready

TiwariAbhishek23 avatar Oct 08 '22 15:10 TiwariAbhishek23

@Nikhil-Ladha had some issues where I can get help ??

SaiRev0 avatar Oct 08 '22 23:10 SaiRev0

@SaiRev0 ask in a public slack channel like #websites channel by tagging appropriate person to whom you would like to ask.

kamalbuilds avatar Oct 09 '22 03:10 kamalbuilds

@Nikhil-Ladha please provide the photos and details of all the collaborating companies so that I can add them

SaiRev0 avatar Oct 11 '22 18:10 SaiRev0

@leecalcote ^^

Nikhil-Ladha avatar Oct 11 '22 18:10 Nikhil-Ladha

For the logo, you could download them from their respective sites docker.com and f5.com

Nikhil-Ladha avatar Oct 11 '22 18:10 Nikhil-Ladha

@Nikhil-Ladha So I have to add only 2 more logos.

So I will add them and shift some from the top so both the rows will have the same numbers

Can we chat some else the comments will go long πŸ˜…

SaiRev0 avatar Oct 11 '22 19:10 SaiRev0

Yes. If you have more doubts ping me on slack.

Nikhil-Ladha avatar Oct 11 '22 19:10 Nikhil-Ladha

image

Does it look fine??

SaiRev0 avatar Oct 12 '22 20:10 SaiRev0

@SaiRev0, it's looking good to me... so long as you're allowing the logos full span of the page (even to the extent that they get cutoff (that's fine).

leecalcote avatar Oct 12 '22 21:10 leecalcote

The significant size difference between the circular Citrix and the circular F5 logos.... hmm. All should be allowed the same space, which means that two circular logos should essentially be of the same size. Cut out any extra padding in the F5 SVG, if needed.

leecalcote avatar Oct 12 '22 21:10 leecalcote

@SaiRev0, it's looking good to me... so long as you're allowing the logos to full span of the page (even to the extent that they get cut off (that's fine).

Ok I will try and adjust the 6 logos to take the full-screen size

And about the logo size I will fix the f5 logo

SaiRev0 avatar Oct 12 '22 21:10 SaiRev0

@SaiRev0 thanks... although, there should need to be a change to the existing logos, I don't believe.

leecalcote avatar Oct 12 '22 21:10 leecalcote

@SaiRev0 thanks... although, there should need to be a change to the existing logos, I don't believe.

So do I have to change some logos (old logos) or leave it as it is?

SaiRev0 avatar Oct 12 '22 21:10 SaiRev0