Skip to content
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

#5894: Blockbase: Fix comment subscription checkboxes #6140

Closed
wants to merge 1 commit into from

Conversation

bobmatyas
Copy link
Contributor

Changes proposed in this Pull Request:

Adjust the grid for post comments on Blockbase to fix overlap issue that happens on Blockbase and children. I tested change on Blockbase, Dorna, and Winkel.

Before

Screenshot on 2022-06-24 at 11-23-18

After

Screenshot on 2022-06-24 at 12-47-22

Related issue(s):

Fixes #5894

@bobmatyas bobmatyas requested a review from a team June 24, 2022 17:13
Copy link
Member

@madhusudhand madhusudhand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. run npm run build include the generated css changes.

  2. I guess it is best to keep the checkboxes in the bottom after the button.
    Following is how other themes block themes show the UI

image

@MaggieCabrera
Copy link
Contributor

I think I'd rather go with #6109 as a solution instead, since it removes all the extra CSS from Blockbase and relies on the site editor instead.

@bobmatyas
Copy link
Contributor Author

I agree it seems like #6109 is a better way to go.

@vcanales
Copy link
Contributor

Closing, as the alternative PR (#6109) was merged.

@vcanales vcanales closed this Jul 14, 2022
@scruffian scruffian deleted the fix/5894 branch July 14, 2022 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dorna, Hari, etc: Comment form labels overlap on mobile
4 participants