Skip to:
Content

BuddyPress.org

Opened 23 months ago

Closed 9 months ago

#8878 closed defect (bug) (wontfix)

spacing issue in responsive

Reported by: kajalgohel's profile kajalgohel Owned by:
Milestone: BuddyPress.org Sites Priority: normal
Severity: normal Version:
Component: Core Keywords: has-screenshots changes-requested has-patch needs-refresh
Cc:

Description

In responsive, the log-in text has a spacing issue.

URL: https://buddypress.org/about/

For better understanding, I provide the Screenshot.

Thanks.

Attachments (4)

CleanShot 2023-05-03 at 17.50.41@2x.png (136.9 KB) - added by kajalgohel 23 months ago.
8878.patch (369 bytes) - added by himshekhar07 23 months ago.
Patch added
8878.1.diff (471 bytes) - added by mikinc860 20 months ago.
8878.2.patch (754 bytes) - added by himshekhar07 20 months ago.
Patch added

Download all attachments as: .zip

Change History (12)

#1 @kajalgohel
23 months ago

  • Keywords needs-patch has-screenshots added

#2 @imath
23 months ago

  • Milestone changed from Awaiting Review to BuddyPress.org Sites

@himshekhar07
23 months ago

Patch added

#3 @mikinc860
20 months ago

  • Keywords changes-requested has-patch added; needs-patch removed

@himshekhar07
I appreciate your efforts but this is removing the bottom padding and looks odd.

you can try this with the below solution

@media screen and (max-width: 782px){
#wpadminbar #wp-toolbar li#wp-admin-bar-bp-login a, #wpadminbar #wp-toolbar li#wp-admin-bar-bp-register a {
    padding: 5px 8px;
}
Last edited 20 months ago by mikinc860 (previous) (diff)

#4 @chiragrathod103
20 months ago

I agree with @mikinc860

#5 @chiragrathod103
20 months ago

  • Keywords needs-refresh added

@mikinc860
20 months ago

@himshekhar07
20 months ago

Patch added

#6 @himshekhar07
20 months ago

Thank you @mikinc860 for the suggestion but without adding any new selector we can try this:

you can try this with the below solution

@media screen and (max-width: 782px) {
    #wpadminbar li#wp-admin-bar-bp-login a,
    #wpadminbar li#wp-admin-bar-bp-register a {padding: 5px 8px !important;}
}

#7 @huzaifaalmesbah
20 months ago

I have tested this patch, and it's working well. Many thanks for providing the solution, @himshekhar07

@media screen and (max-width: 782px) {
    #wpadminbar li#wp-admin-bar-bp-login a,
    #wpadminbar li#wp-admin-bar-bp-register a {padding: 5px 8px !important;}
}

after applying patch result:


https://i.ibb.co/St77yq3/Screenshot-2023-07-29-at-2-58-36-PM.png

#8 @espellcaste
9 months ago

  • Resolution set to wontfix
  • Status changed from new to closed

Closing as per #9193

Note: See TracTickets for help on using tickets.