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

Make the layout sensible #121

Merged
merged 2 commits into from
Jan 23, 2024

Conversation

symbiogenesis
Copy link
Collaborator

@symbiogenesis symbiogenesis commented Aug 9, 2023

The original way that the grid worked involved creating borders between cells by creating empty space, and then using the background color as the border color. That approach is quite counter-intuitive.

This PR wraps all of the cells with their own border. Now everything is way easier to understand for both users and developers.

Fixes #120

@symbiogenesis symbiogenesis force-pushed the sensible-layout branch 2 times, most recently from 05019ff to 41de827 Compare August 9, 2023 23:28
@symbiogenesis symbiogenesis force-pushed the sensible-layout branch 5 times, most recently from cc31ef2 to 3fcdda5 Compare August 10, 2023 03:08
@symbiogenesis symbiogenesis changed the title Make the layout sensible WIP: Make the layout sensible (DO NOT MERGE) Aug 21, 2023
@symbiogenesis symbiogenesis force-pushed the sensible-layout branch 7 times, most recently from b7a6f1e to ee05ea7 Compare January 17, 2024 05:46
@symbiogenesis symbiogenesis changed the title WIP: Make the layout sensible (DO NOT MERGE) Make the layout sensible Jan 17, 2024
@symbiogenesis symbiogenesis changed the title Make the layout sensible WIP: Make the layout sensible Jan 17, 2024
@symbiogenesis symbiogenesis changed the title WIP: Make the layout sensible WIP: Make the layout sensible (DO NOT MERGE) Jan 17, 2024
@symbiogenesis symbiogenesis changed the title WIP: Make the layout sensible (DO NOT MERGE) Make the layout sensible Jan 23, 2024
@symbiogenesis symbiogenesis merged commit 0853f2e into akgulebubekir:main Jan 23, 2024
2 checks passed
@symbiogenesis symbiogenesis mentioned this pull request Mar 11, 2024
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.

BorderColor wrong behaviour
1 participant