fix: hide default weather details (#6665)
This commit is contained in:
1 parent
f46ccadbe4
commit
2570de25a6
4 files changed
+10
-12
No files matched your search
@@ -161,7 +161,7 @@ export interface DefaultWidgetConfig {
|
||||
|
||||
export const defaultWidgetConfigs: DefaultWidgetConfig[] = [
|
||||
{ kind: "clock", width: 2, height: 1 },
|
||||
{ kind: "weather", width: 2, height: 1, options: { showCity: true, hasForecast: true, forecastDayCount: 3 } },
|
||||
{ kind: "weather", width: 2, height: 1, options: { showHumidity: false, showCity: false, hasForecast: false } },
|
||||
{ kind: "bookmarks", width: 2, height: 2, options: { title: "Useful Links", layout: "grid", openNewTab: true } },
|
||||
];
|
||||
|
||||
|
||||
@@ -3012,12 +3012,10 @@
|
||||
"label": "Animate weather icons"
|
||||
},
|
||||
"showHumidity": {
|
||||
"label": "Show humidity",
|
||||
"description": "Display humidity in current conditions, hourly details and daily forecasts"
|
||||
"label": "Show humidity"
|
||||
},
|
||||
"showCurrentWindSpeed": {
|
||||
"label": "Show current wind speed",
|
||||
"description": "Only on current weather"
|
||||
"label": "Show current wind speed"
|
||||
},
|
||||
"useImperialSpeed": {
|
||||
"label": "Use mph for windspeed"
|
||||
|
||||
@@ -69,13 +69,13 @@ describe("weather layout", () => {
|
||||
});
|
||||
|
||||
describe("weather defaults", () => {
|
||||
it("starts rich and static while preserving saved false values", () => {
|
||||
it("starts minimal and static while preserving saved false values", () => {
|
||||
const options = definition.createOptions();
|
||||
expect(options.showCity.defaultValue).toBe(true);
|
||||
expect(options.hasForecast.defaultValue).toBe(true);
|
||||
expect(options.showCity.defaultValue).toBe(false);
|
||||
expect(options.hasForecast.defaultValue).toBe(false);
|
||||
expect(options.forecastDayCount.defaultValue).toBe(3);
|
||||
expect(options.animateIcons.defaultValue).toBe(false);
|
||||
expect(options.showHumidity.defaultValue).toBe(true);
|
||||
expect(options.showHumidity.defaultValue).toBe(false);
|
||||
|
||||
const reduced = reduceWidgetOptionsWithDefinition(definition, {} as never, {
|
||||
showCity: false,
|
||||
|
||||
@@ -26,7 +26,7 @@ export const { definition, componentLoader } = createWidgetDefinition("weather",
|
||||
isFormatFahrenheit: factory.switch(),
|
||||
disableTemperatureDecimals: factory.switch(),
|
||||
animateIcons: factory.switch(),
|
||||
showHumidity: factory.switch({ defaultValue: true, withDescription: true }),
|
||||
showHumidity: factory.switch({ defaultValue: false, withDescription: true }),
|
||||
showCurrentWindSpeed: factory.switch({ withDescription: true }),
|
||||
useImperialSpeed: factory.switch(),
|
||||
location: factory.location({
|
||||
@@ -50,8 +50,8 @@ export const { definition, componentLoader } = createWidgetDefinition("weather",
|
||||
defaultValue: "dddd, MMMM D",
|
||||
withDescription: true,
|
||||
}),
|
||||
showCity: factory.switch({ defaultValue: true }),
|
||||
hasForecast: factory.switch({ defaultValue: true }),
|
||||
showCity: factory.switch({ defaultValue: false }),
|
||||
hasForecast: factory.switch({ defaultValue: false }),
|
||||
forecastDayCount: factory.slider({
|
||||
defaultValue: 3,
|
||||
validate: z.number().min(1).max(7),
|
||||
|
||||
Reference in new issue
Block a user