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.

Sign up for GitHub

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

Jump to bottom

🚀 Optimierungen am Code #51

Merged
merged 57 commits into from
Aug 30, 2022
Merged

🚀 Optimierungen am Code #51

merged 57 commits into from
Aug 30, 2022

Conversation

gnmyt
Copy link
Owner

@gnmyt gnmyt commented Aug 29, 2022 •
edited
Loading

🚀 Optimierungen am Code

Mit dieser PR bereiten wir uns auf den nächsten Release (v1.0.4) vor und optimieren noch ein paar Dinge am Code, bevor die neue Version veröffentlicht wird.

✨ Verbesserungen der UI

  1. Es wird nun eine Info angezeigt wenn kein Test existiert
  2. Das Dropdown-Menü schließt sich jetzt wenn man irgendwo hin klickt
  3. Ein laufender Speedtest zeigt nun einen grünen Puls-Effekt an
  4. Die Mobil-Optimerung wurde verbessert (bis 360px)
  5. MySpeed ist nun PWA-Kompatibel und lässt sich als App installieren (sowohl auf dem Desktop als auf dem Handy)
  6. Das Update-Icon ist nun ein Pfeil nach oben und pulsiert

🐛 Behobene Bugs

  1. Die Deprecation-Warnung wegen dem Paket fontawesome-free-solid wurde behoben
  2. Der Bug in der Speedtest Utility im Server wurde behoben
  3. Der Bug mit dem erneuten anschalten der Tooltips nach geschlossenem Dialog wurde behoben
  4. Ein Bug am Dialog wurde behoben
  5. Es lässt sich nun nicht mehr über einen Tooltip hovern

🔧 Änderungen am Code

  1. Um den Code übersichtlicher zu gestalten wurden alle Requests in ein eigenes RequestUtil verschoben
  2. Einige Komponenten im Client wurden effizienter und übersichtlicher gemacht

Änderungen vorgenommen an ...

  • Server
  • Client
  • Dokumentation
  • Sonstiges: ___

@gnmyt gnmyt marked this pull request as draft August 29, 2022 19:34
@gnmyt
Added the unit to the placeholder in the update methods
ac62cdc
@gnmyt
Fixed a bug in the speedtest util
53afd85
@gnmyt
Created an easier utility to send requests. This cleans some code up
521bfc1
@gnmyt
Removed the deprecation warning in the client
7b282df
@gnmyt
Moved the options of the DropdownComponent.jsx into a separate file
1ee901e
@gnmyt
Moved the dialog infos of the DropdownComponent.jsx into a separate file
b95a693
@gnmyt
Cleaned up the code of the DropdownComponent.jsx
1c21adb
@gnmyt
Moved unsetButtonText -> unsetButton in DialogContext.jsx
ceff778
@gnmyt
Changed the order of the functions in the DropdownComponent.jsx
63ebf17
@gnmyt
Moved SpeedtestComponent.jsx tooltips -> tooltips.js
39a2366
@gnmyt
Moved SpeedtestComponent.jsx errors -> errors.js
5c7334c
@gnmyt
Moved SpeedtestComponent.jsx infos -> infos.js
cd64a0b
@gnmyt
Cleaned up the code in the SpeedtestComponent.jsx
2384d8c
@gnmyt
Moved HeaderComponent.jsx infos -> infos.js
c29e3b0
@gnmyt
Cleaned up the code in the HeaderComponent.jsx
83a8110
@gnmyt
Moved ConfigContext.jsx dialog -> dialog.jsx
4f87104
@gnmyt
Cleaned up the code in the ConfigContext.jsx
3369f6c
@gnmyt
Cleaned up the code in the DialogContext.jsx & fixed the tooltip bug
b5b7698
@gnmyt
Fixed a bug in the DialogContext.jsx
c10d4de
@gnmyt
Added an id to the open dropdown entry in the HeaderComponent.jsx
858feff
@gnmyt
The DropdownComponent.jsx now closes if clicked outside the component
4564e12
@gnmyt
Removed a debug statement in the DialogContext.jsx
c03c079
@gnmyt
Cleaned up the code in the SpeedtestContext.jsx
ae88a2c
@gnmyt
Created a test running animation
86feaea
@gnmyt
A running test now shows a pulse in the header
2964fde
@gnmyt
Added the 192x192 logo to the manifest.json
b4625be
@gnmyt
Added the apple touch icon to the index.html
6211b64
@gnmyt
Added a mobile break div to the LatestTestComponent.jsx
42a4770
@gnmyt
Made the scrollbar smaller on mobile devices
3037af1
@gnmyt
Optimized the mobile version of the Header
b484237
@gnmyt
Optimized the mobile version of the LatestTest
66afbc8
@gnmyt
Optimized the mobile version of the Speedtest
0cf5a66
@gnmyt
Added the vite pwa plugin to the dependencies
f7eaa36
@gnmyt
Added vite pwa to the vite plugins array
2f8bb4e
@gnmyt
Moved LatestTestComponent.jsx dialogs -> dialogs.jsx
f696e77
@gnmyt
Cleaned up the code in the LatestTestComponent.jsx
0fbb803
@gnmyt
Cleaned up the code in the StatusContext.jsx
9561b02
@gnmyt
Cleaned up the code in the StatusContext.jsx
48d28e1
@gnmyt
Cleaned up the code in the styles.sass of the LatestTest
00d40ca
@gnmyt
Upgraded the client dependencies
ad38c32
@gnmyt
Fixed a bug with hoverable tooltips
213bbc9
@gnmyt
Added the cron-parser dependency to the client
bcdbde2
@gnmyt
Added the updateDescription function to the DialogContext.jsx
6d4ca41
@gnmyt
Created the DropdownComponent.jsx utils
69f5bea
@gnmyt
The DropdownComponent.jsx cron now displays the current cron expressi…
ea7eb92 
…on to help the user
@gnmyt
Fixed a bug in the DropdownComponent.jsx
3813639
@gnmyt
Fixed a bug in the LatestTestComponent.jsx
666c498
@gnmyt
Fixed the formatting in the info.jsx
f6cbb7a
@gnmyt
Added the orange hover color
29d8ff7
@gnmyt
Added a custom keyframe for the update icon
a489a24
@gnmyt
Updated the icon & the classes of the update available icon
d434189
@gnmyt
The speedtest icon is now hidden if the status is paused
5c22fed
@gnmyt gnmyt marked this pull request as ready for review August 30, 2022 23:57
@gnmyt gnmyt merged commit eefe42d into development Aug 30, 2022
@gnmyt gnmyt deleted the updates/optimizations branch August 30, 2022 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gnmyt