Skip to content

Commit e925ce6

Browse files
willbreitkreutzthill02krowvin
authored
Feature/map (#222)
* initial commit for maps * styles update * fixed old type * fixed library name * fixed conflicts * Update publish.yml (#276) * run npm audit fix (#275) * 4.0.6 * Update publish.yml (#277) * 4.0.7 * try updated checkout action (#279) * 4.0.8 * Update publish (#280) * try updated checkout action * add repository to package.json? * 4.0.9 * Update publish.yml try and get the npm logs * Update publish.yml * Update publish.yml Remove debug step * 278 corps castle link not redirecting correctly (#281) * Run at /groundwork locally to better emulate docs site * Modify docs template to make home button and breadcrumbs work better. Remove any logic from breadcrumb items, just use the href provided. * Add a `homeUrl` prop to the site wrapper that drives the href of the usace logo in the header, just pass that href down, or if not provided bail with a `#` * add `homeUrl` prop to SiteWrapper docs * 4.1.0 * Fix duplicated docs hash links (#282) * 4.1.1 * try and get up to date package lock * update groundwork-geo * remove annoying type errors * change from console.log to alert, that way we show the user click handler is doable * swap alert for console.log * make mapping navigation links consistent * Fix mapping section of links in nav-links.js * add demo imports for the components being used * fix demo import for GeoJSONLayer * Update map-layout docs Note that it is leftSidebar and the other overlays are ...Toolbar --------- Co-authored-by: thill02 <tessa.j.hill@usace.army.mil> Co-authored-by: Charles Graham <charles.r.graham@usace.army.mil>
1 parent 906a787 commit e925ce6

18 files changed

Lines changed: 1873 additions & 1091 deletions

.eslintrc.cjs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,11 @@ module.exports = {
1919
"off",
2020
{ allowConstantExport: true },
2121
],
22+
"no-unused-vars": ["error", {
23+
"vars": "all",
24+
"args": "after-used",
25+
"ignoreRestSiblings": false,
26+
"argsIgnorePattern": "^_"
27+
}],
2228
},
2329
};

lib/components/display/modal.jsx

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,11 @@ function Modal({
1616
dialogTitle,
1717
dialogDescription,
1818
isStatic = false,
19-
autoFocus=false,
20-
dialogTransition=false,
21-
dialogPanelTransition= false,
19+
autoFocus = false,
20+
dialogTransition = false,
21+
dialogPanelTransition = false,
2222
unmount = true,
23-
role='dialog',
23+
role = "dialog",
2424
buttons,
2525
size = "2xl",
2626
className,
@@ -36,23 +36,23 @@ function Modal({
3636
);
3737
size = "2xl";
3838
}
39-
39+
4040
// Check if Role exists
4141
if (!roleOptions.includes(role)) {
4242
console.error(
43-
`Invalid role option for dialog ${role}. Must be one of: 'dialog','alertdialog'`)
44-
console.warn(
43+
`Invalid role option for dialog ${role}. Must be one of: 'dialog','alertdialog'`,
44+
);
45+
console.warn(
4546
`Defaulting to 'dialog' for role of <Modal modalTitle="${dialogTitle}" .../>`,
4647
);
4748
role = "dialog";
48-
4949
}
5050

5151
return (
5252
<Dialog
5353
open={opened}
5454
onClose={onClose}
55-
static= {isStatic}
55+
static={isStatic}
5656
autoFocus={autoFocus}
5757
transition={dialogTransition}
5858
unmount={unmount}

0 commit comments

Comments
 (0)