Skip to content

feat: confidence boost as a floating stamp. Enhance TierCard recommendation. Pricing calculator#351

Merged
sergiojoker11 merged 22 commits intomainfrom
feat/confidence-boost-and-enhance-tier-card-recommendations-and-tier-selection-calculator
Sep 24, 2025
Merged

feat: confidence boost as a floating stamp. Enhance TierCard recommendation. Pricing calculator#351
sergiojoker11 merged 22 commits intomainfrom
feat/confidence-boost-and-enhance-tier-card-recommendations-and-tier-selection-calculator

Conversation

@sergiojoker11
Copy link
Copy Markdown
Member

@sergiojoker11 sergiojoker11 commented Sep 19, 2025

As title says:

  • confidence boost has gone from tier selection footer to a floating rotated stamp - I got instpired last light like at 3AM xD
  • tierCard now allows 2 recommendation. the default one (earlier behaviour), the recomendation from calculator. It needs to come with a ever-changing-value so react detects it has to render over.
  • The pricing calculator component.

Videos of usages in the corresponding PRs:

…dation so a personalized recommendacion can come from calculator
@sergiojoker11 sergiojoker11 marked this pull request as draft September 19, 2025 14:33
@sergiojoker11 sergiojoker11 changed the title feat: confidence boost as a floating stamp. Enhance TierCard recommen… feat: confidence boost as a floating stamp. Enhance TierCard recommendation. Pricing calculator Sep 19, 2025
return `${minutes} ${translations.min}`;
}
const hours = minutes / 60;
const unitKey = hours === 1 ? 'hour' : 'hours';
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

at shared there is not react-i18n magic

@sergiojoker11 sergiojoker11 force-pushed the feat/confidence-boost-and-enhance-tier-card-recommendations-and-tier-selection-calculator branch from 4f1015d to 27fe59c Compare September 19, 2025 17:20
setCalculationResult(result);
onTierRecommendation({
tierId: result.recommendedTier.id,
trigger: Date.now()
Copy link
Copy Markdown
Member Author

@sergiojoker11 sergiojoker11 Sep 19, 2025

Choose a reason for hiding this comment

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

a static value like "better" or "best" can not be passed alone cause otherwise animations are not trigger when clicking on "Calculate" on the calculator if the result points you at the same tier again.

@sergiojoker11 sergiojoker11 force-pushed the feat/confidence-boost-and-enhance-tier-card-recommendations-and-tier-selection-calculator branch from 657e261 to 06c174f Compare September 19, 2025 17:44
@sergiojoker11 sergiojoker11 marked this pull request as ready for review September 23, 2025 23:59

return (
<Card withBorder className="bg-white max-w-4xl mx-auto" padding="lg" radius="md" shadow="md">
<Group gap="xs" justify="center" mb="md">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Any reason to use <Group> here rather than just a div? Asking because we've talked in the past about Mantine vs Tailwind: Mantine for buttons, input fields, etc and Tailwind for containers and stuff that's for layout.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done

return (
<div className="text-gray-500 py-2 text-center">
<IconCalculator className="mx-auto mb-1 opacity-50" size={24} />
<Text size="xs">{translation.standbyMessage}</Text>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Mantine vs Tailwind for Text (vs just a div/p element).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done

<StandbyDisplayContent lang={lang} />
) : (
<div className="w-full">
<ResultDisplay
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not a big fan of duplicating "HTML" for mobile and desktop tbh... tailwind breakpoints should help us do that kind of stuff.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done

)}
</Button>
{type === 'contact' && (
<Text className="text-gray-600 mt-2" size="xs">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Text: Mantine vs Tailwind

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done

@sergiojoker11 sergiojoker11 merged commit d215c86 into main Sep 24, 2025
2 checks passed
@sergiojoker11 sergiojoker11 deleted the feat/confidence-boost-and-enhance-tier-card-recommendations-and-tier-selection-calculator branch September 24, 2025 12:23
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