profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/benface/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Benoît Rouleau benface Montréal, Canada

benface/tailwindcss-gradients 220

Tailwind CSS plugin to generate gradient background utilities

benface/tailwindcss-filters 161

Tailwind CSS plugin to generate filter and backdrop filter utilities

benface/tailwindcss-children 117

Tailwind CSS plugin to add variants that target child elements

benface/tailwindcss-transforms 104

[DEPRECATED] Tailwind CSS plugin to generate transform utilities

benface/tailwindcss-animations 98

[DEPRECATED] Tailwind CSS plugin to generate animation utilities

benface/tailwindcss-interaction-variants 98

Tailwind CSS plugin to add some missing interaction state variants: checked, group-focus-within, group-active, group-visited, group-disabled, hocus (hover & focus), group-hocus, can-hover, and no-hover

benface/tailwindcss-gap 43

Tailwind CSS plugin to generate gap utilities

benface/tailwindcss-fluid-container 31

Tailwind CSS plugin to add a fluid container component and related utility classes

benface/tailwindcss-layout 17

[DEPRECATED] Tailwind CSS plugin to generate layout utilities

benface/tailwindcss-config 14

Tailwind CSS config generator

push eventbenface/tailwindcss-fluid-container

Mosaad

commit sha a498513b30bee05f0a472d05a539f9aa611293de

ensure that baseStyles are outputted first so they don't override bigger screen styles

view details

Benoît Rouleau

commit sha 40fd23949934d4300f367080e11e2349ded5c2fd

Merge pull request #25 from theMosaad/master Ensure that baseStyles are outputted first

view details

push time in 8 days

PR merged benface/tailwindcss-fluid-container

Ensure that baseStyles are outputted first

I noticed an issue with Tailwind CSS v2 both with Laravel Mix and Nextjs.

The issue is that sometimes the plugin outputs the baseStyles after the @media ones therefore override them.

I can create a minimal reproduction if needed.

+12 -6

1 comment

1 changed file

theMosaad

pr closed time in 8 days

pull request commentbenface/tailwindcss-fluid-container

Ensure that baseStyles are outputted first

Thank you!

theMosaad

comment created time in 8 days

pull request commenttailwindlabs/tailwindcss

Improve preflight v3

@hacknug – Thanks for tagging me here. :) I am very interested in any addition or change that improves the DX of Tailwind out of the box, but I must admit I haven't kept up-to-date with the latest Preflight / modern-normalize changes.

I agree with the rules you posted. Why not throw in letter-spacing: inherit as well, since you have text-align and text-transform?

I came across @sandren's reset a few months back, and I think there are some great ideas in there too: https://gist.github.com/sandren/386527f13b878d9789400b363f1b865e

kripod

comment created time in 8 days

issue commentatomiks/tippyjs

[Feature Request] Add option to prevent opening tooltip on `mouseenter` if mouse button is pressed

@atomiks Ah yes, I suppose returning false if the mouse button is pressed in onShow would do it. I'll try that, thanks!

benface

comment created time in 14 days

push eventwebdna/tailwindcss-aspect-ratio

Benoît Rouleau

commit sha 8b9381a9f5c30a05384a5dcbf545a935ba55d690

Update README.md

view details

push time in 16 days

push eventwebdna/tailwindcss-aspect-ratio

Benoît Rouleau

commit sha 2cddfd6c520a5e5160f936bc1b7b63976205d595

Update README.md

view details

push time in 16 days

issue closedwebdna/tailwindcss-aspect-ratio

css feature

https://developer.mozilla.org/en-US/docs/Web/CSS/aspect-ratio

Why padding?

closed time in 16 days

m2vi

issue commentwebdna/tailwindcss-aspect-ratio

css feature

This plugin was created before the aspect-ratio property was supported by any browser. You should now use the official @tailwindcss/aspect-ratio plugin. I will add a note to the README. Thank you!

m2vi

comment created time in 16 days

issue openedatomiks/tippyjs

[Feature Request] Add option to prevent opening tooltip on `mouseenter` if mouse button is pressed

Problem

I have a range slider with a Tippy on the handle, and on the same page I also have some buttons with Tippys on them. The problem is when the user is dragging the handle and the cursor happens to hover the other buttons, their tooltips open, which is confusing/annoying because the user is not able to click on these buttons while they are dragging the handle.

Solution

I feel like it would be much better UX for the mouseenter trigger to open a tooltip only when the user is not dragging something or selecting text, in other words when the mouse button is not pressed. But because this would be a breaking change (and because customization is good), I would imagine a new config option to get that behavior, which could be called something like noTriggerOnDrag.

created time in a month

issue commentputyourlightson/craft-blitz

Multiple {{ craft.blitz.csrfInput }} should not make multiple XHR requests

Hmm, guys, correct me if I'm wrong. If CSRF tokens remain valid for the user session duration, how would having a single XHR resolve the problem? Theoretically, no matter how many XHRs on the same page, they should all return valid CSRF tokens.

Yeah but no – like Ben says, when it's the user's first visit, there is no session cookie sent with these requests so the session is fully initialized at every request, which renders the previously generated tokens invalid.

benface

comment created time in a month

issue openedatomiks/tippyjs-react

Dynamic `visible` prop doesn't work as expected

I want to sometimes force a tooltip to be visible, but the same tooltip should then revert to "automatic visibility". I do something like visible={forceVisible ? true : undefined}, which seems to work up until forceVisible becomes false again – the tooltip remains visible even though I'm subsequently passing undefined as the visible prop. The only thing that makes it hide is making it false, but then it forces it to be hidden, instead of showing/hiding automatically.

created time in a month

issue commentrails/webpacker

How to build and load for production?

For webpacker to compile the packs in production environment just set NODE_ENV=production before your compiler either ./bin/webpack or ./bin/webpacker --watch

Thank you! If anyone is running into the same issue as I did: I had to disable compile in config/webpacker.yml to prevent rails server from recompiling for development, overwriting the assets built with NODE_ENV=production ./bin/webpack.

moneytree-doug

comment created time in a month

issue commentFosterCommerce/klaviyoconnect

Ability to use the subscribe endpoint when adding users to a Klaviyo list via twig

Added in v4.0.5. Thank you for requesting this!

pepsDev

comment created time in 2 months

release FosterCommerce/klaviyoconnect

4.0.5

released time in 2 months

push eventFosterCommerce/klaviyoconnect

Benoît Rouleau

commit sha ca9fb15fc0963c66948dcb2e73a1421c85fba82d

Release version 4.0.5

view details

push time in 2 months

created tagFosterCommerce/klaviyoconnect

tag4.0.5

A plugin for Craft CMS. Grow your ecommerce business with smarter email automations.

created time in 2 months

push eventFosterCommerce/klaviyoconnect

Benoît Rouleau

commit sha b4d8ee6d3ee9f5f5486d8e82d0b591ccd295bdd9

Ability to use the `subscribe` endpoint when adding users to a Klaviyo list (fix #42)

view details

push time in 2 months

issue closedFosterCommerce/klaviyoconnect

Ability to use the subscribe endpoint when adding users to a Klaviyo list via twig

If users are added to a list via a Twig template, they currently skip the klaviyo double opt-in process. Would it be possible to specify in the template whether the member endpoint or subscription endpoint is used?

See Klaviyo API Docs:

If you would like to subscribe profiles to a list and use the double opt-in settings for the list please use the subscribe endpoint.

closed time in 2 months

pepsDev

issue commentFosterCommerce/shipstation-connect

Invalid Discount price when >= $1000

Fixed in v1.3.6. Thank you reporting this!

dwhoban

comment created time in 2 months

release FosterCommerce/shipstation-connect

1.3.6

released time in 2 months

created tagFosterCommerce/shipstation-connect

tag1.3.6

A plugin for Craft Commerce 2 that integrates with ShipStation.

created time in 2 months

push eventFosterCommerce/shipstation-connect

Benoît Rouleau

commit sha dd25611779cfbb8d7762f162b05e329f2da925be

Release version 1.3.6

view details

push time in 2 months

issue closedFosterCommerce/shipstation-connect

Invalid Discount price when >= $1000

Just received the following from Shipstation:

I understand you are running into an import error message: An error occurred attempting to update orders: Error in XML. Reason: The 'UnitPrice' element is invalid - The value '-1,178.65' is invalid according to its datatype 'http://www.w3.org/2001/XMLSchema:float' - The string '-1,178.65' is not a valid Single value.

We are receiving this error because the unit price for one of your products is invalid. I pulled the order feed for your custom store and found the product causing this issue:

<SKU /> <Name> <![CDATA[TFM]]> </Name> <Quantity>1</Quantity> <UnitPrice>-1,178.65</UnitPrice>

closed time in 2 months

dwhoban

push eventFosterCommerce/shipstation-connect

Benoît Rouleau

commit sha 24881e73389459a3894f32fa8546c9215d46d564

Fix #30 (Invalid Discount price when >= $1000)

view details

push time in 2 months

issue closedFosterCommerce/snipcart-craft-plugin

Return an error rather than rates response in ShippingRateEvent

In ShippingRateEvent events, is it possible to return an error instead of rates, as per https://docs.snipcart.com/v2/webhooks/shipping#error ? (fwiw, it's not as far as I can see. I'd like to do some validation on postcodes, and return an error if no rates because of this)

closed time in 2 months

tobystokes

issue commentFosterCommerce/snipcart-craft-plugin

Return an error rather than rates response in ShippingRateEvent

Added in v1.5.5. Thank you for requesting this!

tobystokes

comment created time in 2 months

delete branch FosterCommerce/snipcart-docs

delete branch : feature/shipping-rate-event-errors

delete time in 2 months

push eventFosterCommerce/snipcart-docs

Benoît Rouleau

commit sha eeea4c3dcdbd5cf64b179d143e505e5e49a024f2

Docs for “Add the ability to return an error in ShippingRateEvent”

view details

Benoît Rouleau

commit sha 0553dd944ee145006e3e7b618b616bb5c51180af

Merge pull request #3 from FosterCommerce/feature/shipping-rate-event-errors Docs for “Add the ability to return an error in ShippingRateEvent”

view details

push time in 2 months

PR merged FosterCommerce/snipcart-docs

Docs for “Add the ability to return an error in ShippingRateEvent”

To go with https://github.com/FosterCommerce/snipcart-craft-plugin/pull/37

+2 -2

0 comment

1 changed file

benface

pr closed time in 2 months