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.

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

feat(trip-form): Advanced Mode Settings Panel #749

Merged

Conversation

amy-corson-ibigroup
Copy link
Contributor

image

@daniel-heppner-ibigroup
Copy link
Contributor

Note removing the old components is a breaking change

@daniel-heppner-ibigroup
Copy link
Contributor

I tried to generate an alpha build but it didn't work :/

Copy link
Contributor

@daniel-heppner-ibigroup daniel-heppner-ibigroup left a comment

Choose a reason for hiding this comment

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

It's great, just a few notes for you

modeSettingValues: {}
};

function pipe<T>(...fns: Array<(arg: T) => T>) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Would you mind moving this function into core utils? We are using it in a lot of places

Copy link
Contributor

Choose a reason for hiding this comment

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

if this feels irrelevant to this PR we can do it later

import { useIntl } from "react-intl";
import SubSettingsPane from "../SubSettingsPane";
import generateModeButtonLabel from "../i18n";
import { invisibleCss } from "..";
Copy link
Contributor

Choose a reason for hiding this comment

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

I think there should be a an empty line between the import groups here?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not getting any errors or warnings here, but it looks like there should be right?

yarn.lock Outdated
version "2.3.1"
resolved "https://registry.yarnpkg.com/classnames/-/classnames-2.3.1.tgz#dfcfa3891e306ec1dad105d0e88f4417b8535e8e"
integrity sha512-OlQdbZ7gLfGarSqxesMesDa5uz7KFbID8Kpq/SxIoNGDqY8lSYs0D+hhtBXhcdB3rcbXArFr7vlHheLk1voeNA==

classnames@^2.3.1:
Copy link
Contributor

Choose a reason for hiding this comment

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

hm, do you know why we got another version of classnames?

@daniel-heppner-ibigroup
Copy link
Contributor

also update snapshots

Copy link
Contributor

@josh-willis-arcadis josh-willis-arcadis left a comment

Choose a reason for hiding this comment

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

amazing!

import { useIntl } from "react-intl";
import SubSettingsPane from "../SubSettingsPane";
import generateModeButtonLabel from "../i18n";
import { invisibleCss } from "..";
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not getting any errors or warnings here, but it looks like there should be right?

@@ -31,6 +33,8 @@ export {
GeneralSettingsPanel,
getBannedRoutesFromSubmodes,
MetroModeSelector,
AdvancedModeSettingsButton,
AdvancedModeSubsettingsContainer,
Copy link
Contributor

Choose a reason for hiding this comment

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

sort components

background-color: ${props => props.accentColor};
color: #fff;
border-bottom-left-radius: ${props => props.subsettings && 0} !important;
border-bottom-right-radius: ${props => props.subsettings && 0} !important;
Copy link
Contributor

Choose a reason for hiding this comment

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

sort css

svg {
height: 26px;
width: 26px;
fill: ${props =>
Copy link
Contributor

Choose a reason for hiding this comment

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

sort css

@josh-willis-arcadis josh-willis-arcadis removed their assignment Jul 2, 2024
BREAKING CHANGE: removes mode subsettings panel

Co-authored-by: Daniel Heppner <[email protected]>
Co-authored-by: Amy Corson <[email protected]>
@amy-corson-ibigroup amy-corson-ibigroup merged commit 0719770 into opentripplanner:master Sep 24, 2024
2 checks passed
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.

3 participants