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

style: reduce company logo size on casestudy/[id] page #2728

Merged
merged 14 commits into from
Mar 11, 2024
Merged

style: reduce company logo size on casestudy/[id] page #2728

Changes from 9 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
3a4f444
style: button and heading margin fix
RamGoel Mar 2, 2024
dc666f6
style: casestudy page for larger screens
RamGoel Mar 3, 2024
33412d6
chore: revert button changes
RamGoel Mar 3, 2024
d98bb1d
chore: image size decrease and indent fix
RamGoel Mar 5, 2024
3342edc
Merge branch 'master' into fix/casestudy
RamGoel Mar 5, 2024
593d120
Merge branch 'master' into fix/casestudy
anshgoyalevil Mar 5, 2024
b33bf94
Merge branch 'master' into fix/casestudy
RamGoel Mar 7, 2024
bf4e70c
chore: remove isAgain
RamGoel Mar 7, 2024
0236881
Merge branch 'master' into fix/casestudy
RamGoel Mar 7, 2024
8e9cde0
chore: remove braces from classname
RamGoel Mar 7, 2024
3034c14
chore: revert spacing changes
RamGoel Mar 8, 2024
feea826
Merge branch 'master' into fix/casestudy
RamGoel Mar 8, 2024
c261acd
Merge branch 'master' into fix/casestudy
RamGoel Mar 10, 2024
4011d6d
Merge branch 'master' into fix/casestudy
anshgoyalevil Mar 11, 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
16 changes: 7 additions & 9 deletions pages/casestudies/[id].js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const renderContent = (content, allComponents, level) => {
return content.map((item) => {
return (
<div
className="mt-10"
className={`mt-10 mx-auto w-11/12`}
Copy link
Member

Choose a reason for hiding this comment

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

Rewrite this in conventional tailwind syntax as it was written previously

Copy link
Contributor Author

@RamGoel RamGoel Mar 7, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

you want to remove mx-auto w-11/12 as well? or just remove {} as there is no condition checking anymore?

Copy link
Member

Choose a reason for hiding this comment

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

Just the braces

key={item.title}
>
<Heading typeStyle={typeStyle} className="mt-8" id={item.title
Expand Down Expand Up @@ -143,9 +143,9 @@ function Index({
wide
>
<div className="max-w-screen lg:flex-row-reverse lg:flex lg:justify-between">
<CaseTOC toc={content} cssBreakingPoint="lg" className="lg:flex-1 bg-blue-100 mt-4 p-4 sticky top-20 overflow-y-auto max-h-screen lg:bg-transparent lg:mt-2 lg:pt-0 lg:pb-8 lg:top-24 lg:max-h-(screen-16) lg:border-l lg:border-gray-200 lg:min-w-[265px] lg:max-w-72 lg:-mr-14" />
<div className="px-4 sm:px-6 xl:px-0 lg:flex-1 lg:max-w-[812px] xl:max-w-5xl case-study">
<div className="mt-10 md:mt-20 flex flex-col md:flex-row justify-between items-center">
<CaseTOC toc={content} cssBreakingPoint="lg" className="lg:flex-1 bg-blue-100 mt-4 p-4 sticky top-20 overflow-y-auto max-h-screen lg:bg-transparent lg:mt-2 lg:pt-0 lg:pb-8 lg:top-24 lg:max-h-(screen-16) lg:border-l lg:border-gray-200 lg:min-w-[200px] lg:max-w-72 lg:-mr-0" />
<div className="xl:max-w-5xl case-study">
<div className=" mt-10 md:mt-20 flex flex-col md:flex-row justify-between items-center w-11/12 mx-auto">
<div className="w-full md:w-[65%]">
<Heading typeStyle="heading-xl" className="countdown-text-gradient">
{casestudy.company.name}
Expand Down Expand Up @@ -189,13 +189,11 @@ function Index({
</Heading>
</div>
</div>
<div className="mt-10 md:mt-0">
<img
<img
src={casestudy.company.logo}
alt={casestudy.company.name}
className="w-[350px] rounded-lg"
className="w-[250px] mt-5 md:mt-0 mx-auto rounded-lg"
/>
</div>
</div>
{renderContent(content, allComponents, 0)}
</div>
Expand All @@ -204,4 +202,4 @@ function Index({
);
}

export default Index;
export default Index;
Loading