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

Fix: enterprise page #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Jericho-Tsalis
Copy link

@Jericho-Tsalis Jericho-Tsalis commented Jul 16, 2024

Removed the shadow-lg and dark:shadow-slate-800 classes from images on lines 79, 85, and 93 respectively.

Closes #34

@Jericho-Tsalis Jericho-Tsalis force-pushed the Jericho-Tsalis-patch-1-enterprise.vue branch 2 times, most recently from 253973f to 39f117a Compare July 19, 2024 07:21
@Jericho-Tsalis Jericho-Tsalis changed the title Update enterprise.vue Fix: enterprise page Jul 19, 2024
fix: removed the `shadow-lg` and `dark:shadow-slate-800` classes from three images.
@Jericho-Tsalis Jericho-Tsalis force-pushed the Jericho-Tsalis-patch-1-enterprise.vue branch from 39f117a to a731082 Compare July 19, 2024 08:08
@romangg
Copy link
Member

romangg commented Jul 19, 2024

Removing the shadow won't look bad with the icons now free-floating around on the page?

Otherwise can we replace the shadow with something else?

@Jericho-Tsalis
Copy link
Author

Jericho-Tsalis commented Jul 19, 2024

Removing the shadow won't look bad with the icons now free-floating around on the page?

Otherwise can we replace the shadow with something else?

Another option is using vector images and thedrop-shadow class. I'll look into that.

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.

Box shadow artifacts
2 participants