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.

Sign up for GitHub

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

Making our CTA buttons the same v2 #141

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
+6 −6
Open

Making our CTA buttons the same v2 #141

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2ddc4d3
wip
MrGKanev Jan 14, 2024
51cd778
compiled
MrGKanev Jan 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion public/themes/bimbala/css/app.css
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion public/themes/bimbala/mix-manifest.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{
"/js/app.js": "/js/app.js?id=31044e78bdf2a1f38951",
"/css/app.css": "/css/app.css?id=097de9a4776dd9d01d8a"
"/css/app.css": "/css/app.css?id=1cfc3e1d36292ca6c115"
}
4 changes: 2 additions & 2 deletions resources/views/themes/bimbala/home.blade.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
<h1 class="text-4xl font-bold text-white lg:text-6xl xl:text-6xl">{{ theme('home_headline') }}</h1>
<p class="text-base text-blue-200 lg:text-xl">{{ theme('home_subheadline') }} {{-- - {{ theme('home_description') }} --}}</p>
<div class="flex flex-col items-center w-full mb-10 sm:flex-row sm:mb-20">
<a href="{{ theme('home_cta_url') }}" class="w-full px-8 py-3 text-lg text-center bg-purple-900 text-white transition duration-150 ease-in-out sm:w-auto focus:outline-none rounded">Demo </a>
<a href="https://calendly.com/bimbalacom/30min" class="w-full px-8 py-3 mt-5 ml-3 text-lg text-center text-white transition duration-150 ease-in-out bg-transparent border border-white sm:ml-6 sm:w-auto sm:mt-0 focus:outline-none hover:bg-white hover:text-blue-700 rounded">Book a demo</a>
<a href="{{ theme('home_cta_url') }}" class="w-full px-8 py-3 mb-6 text-white text-center bg-purple-600 border border-transparent border-solid rounded-md select-none sm:mb-0 sm:w-auto hover:bg-purple-700 hover:border-purple-700 hover:text-white">Demo</a>
Copy link
Member

Choose a reason for hiding this comment

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

It looks too bright to me. Let's get back to the old purple opacity.

<a href="https://calendly.com/bimbalacom/30min" class="w-full px-8 py-3 text-center sm:ml-6 sm:w-auto sm:mt-0 text-white no-underline align-middle bg-transparent border-2 border-white border-solid rounded-full cursor-pointer select-none md:w-auto lg:px-7 xl:px-10 hover:border-white hover:bg-white hover:text-purple-700 focus:shadow-xs focus:no-underline">Book a demo</a>
</div>
</div>
<div class="relative flex items-center justify-center w-full mt-10 md:mt-0 md:w-1/2">
Expand Down
4 changes: 2 additions & 2 deletions resources/views/themes/bimbala/partials/cta.blade.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,10 @@
</div>
</div>
<div class="relative flex flex-col w-full px-4 mt-10 leading-6 text-left md:flex-row lg:w-5/12 xl:w-1/2 justifty-end">
<a href="https://board.bimbala.com/" class="inline-flex items-center justify-center w-full h-16 px-10 py-0 mb-8 text-xl font-semibold text-center text-white no-underline align-middle bg-green-400 border border-transparent border-solid rounded-full cursor-pointer select-none md:mb-0 md:mr-8 md:w-auto lg:px-7 xl:px-10 hover:bg-green-500 focus:shadow-xs focus:no-underline">
<a href="https://board.bimbala.com/" class="inline-flex items-center justify-center w-full h-16 px-10 py-0 mb-8 text-xl font-semibold md:mb-0 md:mr-8 md:w-auto lg:px-7 xl:px-10 text-white text-center bg-purple-600 border border-transparent border-solid rounded-md select-none sm:mb-0 sm:w-auto hover:bg-purple-700 hover:border-purple-700 hover:text-white">
MrGKanev marked this conversation as resolved.
Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

The purple doesn't stand out on purple background here.

Our Board
</a>
<a href="{{ route('register') }}" class="inline-flex items-center justify-center w-full h-16 px-10 py-0 text-xl font-semibold text-center text-white no-underline align-middle bg-transparent border-2 border-purple-600 border-solid rounded-full cursor-pointer select-none md:w-auto lg:px-7 xl:px-10 hover:border-white hover:text-white focus:shadow-xs focus:no-underline">
<a href="{{ route('register') }}" class="inline-flex items-center justify-center w-full h-16 px-10 py-0 text-xl font-semibold text-center text-white no-underline align-middle bg-transparent border-2 border-white border-solid rounded-full cursor-pointer select-none md:w-auto lg:px-7 xl:px-10 hover:border-white hover:bg-white hover:text-purple-700 focus:shadow-xs focus:no-underline">
MrGKanev marked this conversation as resolved.
Show resolved Hide resolved
Get started
</a>
</div>
Expand Down