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

style: align Heading and Items #6988

Merged
merged 20 commits into from
Feb 1, 2024
Merged

Conversation

olegfedak
Copy link
Contributor

Summary
It's just a small fix of styles without changing the design.

As a result
decap-style-fix

Copy link

netlify bot commented Nov 23, 2023

‼️ Deploy request for cms-demo rejected.

Name Link
🔨 Latest commit afbb629

Copy link

netlify bot commented Nov 23, 2023

Deploy Preview for decap-www canceled.

Name Link
🔨 Latest commit 2cb0035
🔍 Latest deploy log https://app.netlify.com/sites/decap-www/deploys/65bb9325b4d1d000075abc41

martinjagodic
martinjagodic previously approved these changes Nov 23, 2023
@martinjagodic martinjagodic enabled auto-merge (squash) November 23, 2023 15:31
auto-merge was automatically disabled November 23, 2023 19:36

Head branch was pushed to by a user without write access

martinjagodic
martinjagodic previously approved these changes Nov 24, 2023
@martinjagodic martinjagodic enabled auto-merge (squash) November 24, 2023 08:43
auto-merge was automatically disabled November 28, 2023 22:46

Head branch was pushed to by a user without write access

Copy link
Contributor Author

@olegfedak olegfedak left a comment

Choose a reason for hiding this comment

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

As a result. Part 2
Знімок екрана 2023-11-24 о 14 55 14

martinjagodic
martinjagodic previously approved these changes Nov 29, 2023
Copy link
Member

@martinjagodic martinjagodic left a comment

Choose a reason for hiding this comment

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

@olegfedak this looks nice, thanks! If you indend to add more changes to this PR please convert it to draft. It doesn't make much sense to review every commit :)

@martinjagodic martinjagodic enabled auto-merge (squash) November 29, 2023 08:29
@olegfedak olegfedak marked this pull request as draft November 29, 2023 15:08
auto-merge was automatically disabled November 29, 2023 15:08

Pull request was converted to draft

@olegfedak
Copy link
Contributor Author

olegfedak commented Nov 29, 2023

@martinjagodic oh, I will! It seems I didn't completely understand how the pull request works. I might should work in a branch from the forked repo master

@martinjagodic
Copy link
Member

That's ok. Mark it as ready for review when you're done.

@olegfedak olegfedak changed the title style: align SidebarHeading with TopHeading style: align Heading Dec 1, 2023
Copy link
Contributor Author

@olegfedak olegfedak left a comment

Choose a reason for hiding this comment

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

Result:
Знімок екрана 2023-12-02 о 00 34 06

@olegfedak olegfedak marked this pull request as ready for review December 1, 2023 23:31
@olegfedak olegfedak changed the title style: align Heading style: align Heading and Items Dec 1, 2023
martinjagodic
martinjagodic previously approved these changes Dec 7, 2023
@martinjagodic martinjagodic enabled auto-merge (squash) December 7, 2023 08:22
auto-merge was automatically disabled December 14, 2023 20:33

Head branch was pushed to by a user without write access

@martinjagodic martinjagodic enabled auto-merge (squash) January 8, 2024 08:44
dev-test/config.yml Outdated Show resolved Hide resolved
auto-merge was automatically disabled January 18, 2024 12:52

Head branch was pushed to by a user without write access

@olegfedak olegfedak requested a review from a team as a code owner February 1, 2024 12:48
@martinjagodic martinjagodic enabled auto-merge (squash) February 1, 2024 12:49
@martinjagodic martinjagodic merged commit 84ea660 into decaporg:master Feb 1, 2024
16 checks passed
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.

2 participants