-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[PT Run][New Plugin] Time zone plugin #11431
Conversation
We could user default time symbol or we may create a new icon form clock and globe. |
@TobiasSekan
|
@TobiasSekan |
f090dc6
to
67f2613
Compare
i like this plugin very much |
I have seen that I can't take only the first letters of the name of a Time-Zone to match the Time-Zone shortcut (like UTC), so I will add a separate JSON to match all possible shortcuts and the official cities of the time zones. Stay tuned BTW: What should I do about cities that have more than one Time-Zones? Any Ideas? |
My opinion is just leverage basic date time shifts. Time zone stuff is NP hard problem IMO |
@niels9001 Can you make a light and dark symbol for the timezone plugin? |
Will do! |
|
||
var stringBuilder = new StringBuilder(); | ||
|
||
stringBuilder.Append(Resources.Name).Append(':').Append(' ').AppendLine(timeZone.Name); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If possible please use the suggested syntax which is easier to read:
stringBuilder.Append(Resources.Name).Append(':').Append(' ').AppendLine(timeZone.Name); | |
stringBuilder.Append($"{Resources.Name}: {timeZone.Name}"); |
@TobiasSekan do you need anything from us to wrap this up? |
b0499be
to
8e84c2b
Compare
Sorry for long down time, I will work the next days on this draft to bring it into PR state, happy holidays @crutkas You can checkout the draft an test the search results and I need a new symbol from @niels9001 |
4698389
to
518d98f
Compare
@check-spelling-bot Report🔴 Please reviewSee the files view or the action log for details. Unrecognized words (6)MCDT Previously acknowledged words that are now absentChaseKnowlden CleanCodeDeveloper CTLCOLORSTATIC Deuchert efgh errc FIXME Grayscale hmon iccex ICONINFORMATION IMonitor INITCOMMONCONTROLSEX INSTALLLOGATTRIBUTES INSTALLLOGMODE INSTALLUILEVEL LASTEXITCODE MAINICON MAKELPARAM msiexec MSIINSTALLER MSIL NATIVEFNTCTL netlify pdb Pui Qin rdeveen rexit SETRANGE SETSTEP spdth sregex STEPIT symlink taskkill toolset UITo uninit utc We'd wifstreamTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:TobiasSekan/PowerToys.git repository
If the flagged items do not appear to be textIf items relate to a ...
|
src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.TimeZone/plugin.json
Show resolved
Hide resolved
@TobiasSekan |
There is a
|
@check-spelling-bot Report🔴 Please reviewSee the files view or the action log for details. Unrecognized words (3)Sring Previously acknowledged words that are now absentabap abcd abcdefgh appcontainer atop autogenerates AZCLI azurecr binskim bios btm buildcommand buildtools cameligo CCB cdpxwin cdxml CEBB CFFDAFADA ChaseKnowlden cjs CleanCodeDeveloper cljs CPPARM CPPx crutkas csx CTLCOLORSTATIC defaultcommand Deuchert dupenv DUSTIN efgh errc errorlevel estdir etcore Firefox FIXME fsscript Gamebar graphql Grayscale gsuberland hmon iccex ICONINFORMATION IMonitor INITCOMMONCONTROLSEX INSTALLLOGATTRIBUTES INSTALLLOGMODE INSTALLUILEVEL Javascript julia Knowlden kotlin ktm kts LASTEXITCODE LEQ ligo linkid litcoffee MAINICON MAKELPARAM Minimizeallwindows mkdir moderncop msiexec MSIINSTALLER MSIL namings NATIVEFNTCTL neq netlify nocache php phps plx policheck popd powerquery psc psm Pui pushd pyc pyd pyi pyo pyz Qin rda rdata rdeveen rds reportbug rexit robocopy scm SETRANGE SETSTEP SHAREIMAGELISTS smallbasic spamming spdth sregex STEPIT stx svh SWITCHEND symlink symlinks systemverilog taskkill tbc tcl timezone trackpad typeparamref UITo umd uninit UTC UWPUI vbhtml verilog vse vsix WDK wdksetup wdkvsix We'd webclient webpack wifstream WINMSAPP WTLTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:TobiasSekan/PowerToys.git repository
If the flagged items do not appear to be textIf items relate to a ...
|
I have fix the search |
...modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.TimeZone/Classes/TimeZoneSettings.cs
Outdated
Show resolved
Hide resolved
|
||
var settingsValue = option?.Value | ||
?? GetAdditionalOptions().FirstOrDefault(x => x.Key == name)?.Value | ||
?? default; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is default? true or false ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By the way I think we can use .First()
instead of .FirstOrDefault() ?? Default
because the default value must be there. If it is not there, the setting isn't defined and the plugin code is buggy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is default? true or false ?
is always false
, same as default(bool)
@check-spelling-bot Report🔴 Please reviewSee the files view or the action log for details. Unrecognized words (1)timezones Previously acknowledged words that are now absentabap abcd abcdefgh appcontainer atop autogenerates AZCLI azurecr binskim bios btm buildcommand buildtools cameligo CCB cdpxwin cdxml CEBB CFFDAFADA ChaseKnowlden cjs CleanCodeDeveloper cljs CPPARM CPPx crutkas csx CTLCOLORSTATIC defaultcommand Deuchert dupenv DUSTIN efgh errc errorlevel estdir etcore Firefox FIXME fsscript Gamebar graphql Grayscale gsuberland hmon iccex ICONINFORMATION IMonitor INITCOMMONCONTROLSEX INSTALLLOGATTRIBUTES INSTALLLOGMODE INSTALLUILEVEL Javascript julia Knowlden kotlin ktm kts LASTEXITCODE LEQ ligo linkid litcoffee MAINICON MAKELPARAM Minimizeallwindows mkdir moderncop msiexec MSIINSTALLER MSIL namings NATIVEFNTCTL neq netlify nocache php phps plx policheck popd powerquery psc psm Pui pushd pyc pyd pyi pyo pyz Qin rda rdata rdeveen rds reportbug rexit robocopy scm SETRANGE SETSTEP SHAREIMAGELISTS smallbasic spamming spdth sregex STEPIT stx svh SWITCHEND symlink symlinks systemverilog taskkill tbc tcl timezone tlbimp trackpad typeparamref UITo umd uninit UWPUI vbhtml verilog vse vsix WDK wdksetup wdkvsix We'd webclient webpack wifstream WINMSAPP WTLTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:TobiasSekan/PowerToys.git repository
If the flagged items do not appear to be textIf items relate to a ...
|
Hi @TobiasSekan , Please review and merge it or apply the changes to your branch? |
PR11431 fix GitHub errors
Thank you 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM! Tried it and works well.
Thank you for the contribution!
* Initial commit - simple idea for a time zone plugin * Translations, better search results, copy to clipboard, cleanup * fix typo * Add shortcut search and prepare JSON for later usage * Fix typo * Use timezone Data only from JSON * Exclude json file from spell checker * fix wrong dst * Improved results (title, subtitle, tooltip) and fix namespace/class problem * Always show full offset (-##:## and +##:##) * Add and show timezone names (first pass) * Fix typos * fix build * JSON: fix wrong minus sign and put extra country info the end * Improved Subtitle for many matched countries and allow full offset search (+ and -) * Allow more than one names for time zones and remove leftover * Add military time zone names, and fix name result * Only use one JSON entry for one time zone * Use TimeSpan for offset, use build-in calculation for time in time zone * add descriptions for JSON schema * Fix typos * Split out names in separate properties * Add many time names, time zone names and shortcuts * Add additional options and most code documentation * Fix unreadable TimeSpans in JSON and rename helper class * Fix not allowed commas in JSON file * Cut to long time and time zone names in title * Fix missing results for names and offsets * Better result and show only one result when offset are identical (respect daylight saving time) * Show generic name fot time zones without names * Typo fixes * Fix not working serach by shortcuts * Fix german resx file -> english resx file * Translate all names and countires * Fix not working context menu * Typo fixes, fix wrong shortcut in names, comments, few better variable names * New symbols - thx to niels9001 * Search by shortcuts for time names * update schema * Add more time zone names and shortcuts (second pass), make spell checker happy * Reduce matching checks * Show shortcuts in tool-tips, avoid string converting * Show only names that match the query * Make all translatable (Part 1) * Make all translatable (part 2 of 2) * XML Doc * Fix plugin name (type) * Fix Typos * Add TimeZone Plugint to WXS * Add TimeZone plugin to sign pipeline * Add Documentation * Remove double spell entries * Remove TODO leftovers * Fix for results with no countries * Fix typos * fix typos * Fix broken siolution after rebase * Update target framework to make build happy * fix wrong guid count in WXS * fix wrong output folder (setup wasn’t found files) * Address feedback from @jsoref - fix spell check * typo fix - one leftover in expect.txt * Switch to .NET6 and update dokumentation * Address feedbacks, and fix search bug * fix installer build error * fix spellchecker * Address feedback from @htcfreek Co-authored-by: Sekan, Tobias <tobias.sekan@axp-consulting.de>
in what way then? |
No, this plugin provide only the possibility to find time zones and there current time by
|
Can I ask, why there is no Ukraine, but there is Transnistria, DPR, Abkhazia and other not recognized countries? Picture |
why is there no ukraine? |
b5d47f161757c8c93f8645149fa1e64600648824a3ee50066efa95d41e0ce98db2deec01e4a9afbf09a2d364ae8d561ee737a9ba08a8f7cd488557c9a82f3b20.1.mp4 |
Was this plugin removed from PowerToys? How do I install this plugin on the most recent release? |
hi, is this plug-in still part of PowerToys, and how do I access it if so |
Summary of the Pull Request
What is this about:
Time Zone plugin for PT Run
Features:
Kamchatka
,Prince Edward Island
,France
Pacific
,Mountain
,Central
WEST
,UTC
,PST
-12:00
,-7
,5
,9:30
What is include in the PR:
How does someone test / validate:
Manual testing
Quality Checklist
Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.