#6631 closed defect (bug) (fixed)
#profile-nav buttons bug
Reported by: | daniluk4000 | Owned by: | djpaul |
---|---|---|---|
Milestone: | 2.4 | Priority: | normal |
Severity: | normal | Version: | 2.3.3 |
Component: | Core | Keywords: | has-patch |
Cc: | daniluk4000@… |
Description
On User Profile Edit page when the number of notifications => 4 see screenshot
Attachments (3)
Change History (15)
This ticket was mentioned in Slack in #buddypress by daniluk4000. View the logs.
9 years ago
#3
@
9 years ago
- Keywords has-patch removed
has-patch
keyword is used when a 6631.patch
file is attached, which is actually an SVN patch.
#5
@
9 years ago
I believe, RTL file is auto-generated, some of the core devs will (or not) correct me in this.
Anyway, now it's much easier for core devs to apply your patch :)
And that's not just my wish, that's something I was told years ago when started submitting my first patches here too.
#7
@
9 years ago
Thanks for the patch @daniluk4000 , looks like a good spot.
Not looked at this but did you check whether the parent element doesn't just need clearing. My concern with a fixed pixel padding would be if more admin notices appeared, does the padding keep things below any number of possible notices?
As for the rtl files they will be generated by the grunt task when the patch is committed.
#10
@
9 years ago
I think this is your first patch contribution to BuddyPress, daniluk4000! Thanks very much.
How do I fix it: