profile
viewpoint

rowanmanning/proclaim 66

A simple assertion library for server and client side JavaScript.

Financial-Times/o-editorial-typography 1

Typography styles for editorial content

Financial-Times/origami-migration-guru 1

Find and explore component dependents to plan major release migrations.

Financial-Times/origami-whitelabel-brand-demo 0

A temporary project to demonstrate customised Origami components, based on the whitelabel brand.

notlee/audrey 0

A simple single-user and self-hosted feed reader

notlee/karma-scss-preprocessor 0

Karma preprocessor to compile SCSS on the fly with node-sass

notlee/style-dictionary 0

A build system for creating cross-platform styles.

notlee/tweet-delete 0

A lambda to delete tweets and likes older than a defined age

push eventFinancial-Times/origami

Lee Moody

commit sha 52ecd1b33e4ec827cd61cb40bc6b460f6546935c

backstage: correct style dictionary reference

view details

push time in 5 days

push eventFinancial-Times/origami

Lee Moody

commit sha 4f3063a840dd97970a94b16451f97cabc3af08ee

backstage: update o-buttons high contrast fallback for design tokens experiment

view details

push time in 6 days

push eventFinancial-Times/origami

Lee Moody

commit sha 66e127eb7365fdc7bba6d095f56bc7983d010ae0

backstage: Add to buttons design token demo with dark mode support throw awway attempts to make the api nicer and bundle size smaller for now

view details

push time in 6 days

PullRequestReviewEvent
PullRequestReviewEvent

pull request commentFinancial-Times/origami

fix!: Drop IE9 support in o-stepped-progress

Don't forget the readme! 😄 😬 https://github.com/Financial-Times/origami/blob/2e057208699c5d390becce11c641e9afd35a0eda/components/o-autocomplete/README.md?plain=1#L275

chee

comment created time in 6 days

pull request commentFinancial-Times/origami

fix!: Drop IE9 support in o-stepped-progress

I don't think it is because I don't think we ever told anyone we support IE9 and this component was made after we no longer supported IE9

I think you're right on that and would be confident releasing this as a patch, but if you'd prefer to be cautious could you add a migration guide to go along with the major release?

chee

comment created time in 6 days

pull request commentFinancial-Times/origami

fix!: Drop IE9 support in o-stepped-progress

I don't think it is because I don't think we ever told anyone we support IE9 and this component was made after we no longer supported IE9

Plus in the past we've dropped IE8/9/10 support without releasing a breaking change, e.g o-grid. Maybe technically a no-no but I'd be in favour of continuing that trend and making this a patch release too, for the smoothest upgrade path for Origami users.

Maybe our promise going forward should be "we support what the ft supports" rather than a specific browser, unless our components are ever generic enough to be helpful to wider open source users...

chee

comment created time in 6 days

created tagFinancial-Times/ft-date-format

tagv2.1.0

created time in 7 days

release Financial-Times/ft-date-format

v2.1.0

released time in 7 days

push eventFinancial-Times/origami

Lee Moody

commit sha dd9340a4fdb530d5be90daee58ad5098e82c8965

backstage: add todo notes for o-buttons/design token experiment

view details

push time in 10 days

push eventFinancial-Times/origami

Alice Bartlett

commit sha f81547c3a588796440c6cd37ff4d9e73a34b1caf

Merge pull request #9 from Financial-Times/focus-style Focus style

view details

Rowan Manning

commit sha fe54b7a308d0c68e2598fcd49bc94519d9d2d1a4

Add support contact details to origami.json (#10)

view details

Gabrielle von Koss

commit sha 360505ef5242be2528b09d8d740316f48b0de755

Design updates (#12) * reduces type size and read more bar height * change toggle position in demo + readme * remove background colours * change markup * rearranged markup/list syling * correct margins and padding(s) * clean up colours * add b2b card * change expander functionality * remove images * add migration guide * typo * adds flexbox container for all equal heights * add sneaky design change from james * alignment nudges * update readme * linting ☠🙄 * flex browser support * stray commented hmtl * remove unnecessary padding n such * double props 🤦‍♀️ * use mixin :muscle: * more bumps for flex fallback ie9 * more padding/margin corrections * missed px values * adds a little mustard cut

view details

Lee Moody

commit sha 36d8e06b5fac4f2ba11fb16277eb80f22a1a2185

micro README changes (#14)

view details

notlee

commit sha b2ce274f2e179aa843dd9c19a11f651c66473715

Add missing demo titles.

view details

Lee Moody

commit sha c8a11d572b73a27db8d057b0ce46841568c5d7d3

Merge pull request #15 from Financial-Times/missing-demo-titles Add missing demo titles.

view details

notlee

commit sha f6167325a774b43b7e42c6f0f2fc8d797e6efc4c

Update use of o-buttons.

view details

Lee Moody

commit sha dcac26cdca5c356593aa257f00c756e80fe75167

Merge pull request #16 from Financial-Times/update-o-button-use Update use of o-buttons. Fix focus state.

view details

Liam Keaton

commit sha c1c92f43fc0b77c4288c91d2e88a61a826d796a8

Reordering price and adjusting padding

view details

Remy Bach

commit sha 81db85df92d7fc41dcb810f6c5fec89674f5216e

Calc height to apply to all cards in a container

view details

Gabrielle von Koss

commit sha 6e8dd37e5121c8037a4359bddab997093bce4a0b

update support details

view details

Remy Bach

commit sha ae620228f553221f069f00d13d7c1fe8f7ee27b1

Demystify magic number

view details

Rémy Bach

commit sha 4094c9be8255356dc4977f48789e271af86d8672

Merge pull request #18 from Financial-Times/lk/price-order-change Price order change & design updates.

view details

Liam Keaton

commit sha c0afddbe296d82079c0854be98d06dec8a0ced9c

Implementing a two column layout

view details

Liam Keaton

commit sha b535d27f21c56697e816b2b8b307a0e4e3506339

Merge pull request #19 from Financial-Times/lk/two-column Implementing a two column layout

view details

Rémy Bach

commit sha c9137f27f558b87b219b0f3b9755cd38bcefe87e

Update Conversion email address. (#22)

view details

Rowan Manning

commit sha 1dca850672d0cee63682dfda6f99023feb82e3f7

Switch to CircleCI 2 (#20)

view details

Rémy Bach

commit sha 0cb9627d683e4c2086bf33d5b735f903b9528787

Add swg & currency mismatch stuff (#21)

view details

Jennifer Kerr

commit sha a4387094bea65b3f2c4d229f248d1e08e0bdd107

update primary button to standard format

view details

Jennifer Kerr

commit sha d87836885a6cba8bd8053313d794517a0bbe7ca6

also make change for discount card button

view details

push time in 10 days

Pull request review commentFinancial-Times/origami

make o-no work with the * range and exact versions

 #!/usr/bin/env node  import process from 'node:process'-import {readPackageAsync} from "read-pkg"-import semverExtra from "semver-extra"-import origamiComponents from "./last-versions-available-on-bower.js"-import majors from "major-versions"-import isBounded from "semver-bounded"-import dedentTabs from "dedent-tabs"+import {readPackageAsync} from 'read-pkg'+import semverExtra from 'semver-extra'+import origamiComponents from './last-versions-available-on-bower.js'+import majors from 'major-versions'+import isBounded from 'semver-bounded'+import exactVersion from 'exact-version'+import dedentTabs from 'dedent-tabs' const dedent = dedentTabs.default  function checkDependencies(dependencies) { 	const dependenciesToDowngrade = [] 	if (dependencies) {-		for (const [name, versionRange] of Object.entries(dependencies)) {+		for (let [name, versionRange] of Object.entries(dependencies)) {+			versionRange = versionRange.trim() 			if (name in origamiComponents) { 				// Versions which end with `-bower` are only on bower and the version on npm which  				// is compatible will be the same version number but without `-bower` on the end. 				const lastVersionAvailableOnBower = origamiComponents[name].replace('-bower', '')-				const isUnbounded = !isBounded.range(versionRange)+				const isUnbounded = versionRange === '*' || !isBounded.range(versionRange) 				if (isUnbounded) {-					dependenciesToDowngrade.push([name, lastVersionAvailableOnBower])-				+					dependenciesToDowngrade.push([name, lastVersionAvailableOnBower, versionRange]) 				} else {-					const versions = [lastVersionAvailableOnBower, ...majors(versionRange)]-					const maxVersion = semverExtra.max(versions)+					const isExactVersion = exactVersion(versionRange)+					if (isExactVersion) {+						if (semverExtra.gt(versionRange, lastVersionAvailableOnBower)) {+							dependenciesToDowngrade.push([name, lastVersionAvailableOnBower, versionRange])+						}+					} else {+						const versions = [lastVersionAvailableOnBower, ...majors(versionRange)].map(version => {+							return String(semverExtra.coerce(version, {+								loose: true+							}))+						})+						console.log({versions},majors(versionRange))

I think this console log is there accidentally?

JakeChampion

comment created time in 11 days

PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent

push eventFinancial-Times/origami

Fox

commit sha 021fe31b5097f1a63c1084b79afd4e744382cf67

chore: release main

view details

push time in 18 days

delete branch Financial-Times/origami

delete branch : release-please/branches/main

delete time in 18 days

PR merged Financial-Times/origami

chore: release main autorelease: pending

:robot: I have created a release *beep* *boop*

<details><summary>eslint-config-origami-component: 2.2.0</summary>

Features

  • add a new and improved jsdoc plugin (0ec179c)

Bug Fixes

  • make origami-component eslint be able to parse async functions and other modern syntax (33a14af) </details>

This PR was generated with Release Please. See documentation.

+15 -2

0 comment

3 changed files

origamiserviceuser

pr closed time in 18 days

push eventFinancial-Times/origami

Jake Champion

commit sha 5aa7f55b4cad89e0bb89ee4aff2658902a1a64d7

fix: stop building specific demos for each brand in percy this is all handled within obt demo now instead, which is great

view details

chee

commit sha ee3401fa6d67a2eff0a7fe95692b8425a223884f

docs: update versions and migration links

view details

chee

commit sha f46d04059c27688e7b64af4b5de12308649c945e

docs: note the -bower versions

view details

chee

commit sha 799c370a6eb7a26a5e1290e6f31f322ee624c0a8

Apply suggestions from code review

view details

Lee Moody

commit sha 06509bac442a762c0097a510ebce3a49086c7c69

Merge branch 'main' into release-please/branches/main

view details

push time in 18 days

PullRequestReviewEvent

pull request commentFinancial-Times/origami

Allow configuring the path to the manage cookies page

Would it be possible to add docs to the readme?

Agree this component should be less tied to implementation 👍 as we chatted about we could create a component to provide a UI without assumption around how consent is managed

chee

comment created time in 19 days

issue commentFinancial-Times/origami

Proposal: Design Tokens (WIP)

Ooops, @todo just get a few notifications

notlee

comment created time in 20 days

PullRequestReviewEvent

Pull request review commentFinancial-Times/origami-image-set-tools

mark the image-set-tools as dead since image-sets now live within the image-service itself

 Origami Image Set Tools ======================= +**No Longer used - ImageSets now live within the [Origami Image Service](Now that the image-set lives within the origami-image-service (Financial-Times/origami-image-service#763) - we no longer need this tool at all)**

the markdown for the link here isn't correct, maybe this?

**No Longer used - ImageSets now live within the [Origami Image Service](Financial-Times/origami-image-service#763)**
JakeChampion

comment created time in 20 days

PullRequestReviewEvent

create barnchFinancial-Times/origami

branch : nov-2021-newsletter

created branch time in 21 days

more