Add Open-Meteo provider and UI #227
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1607,6 +1607,19 @@ <h1>Material You New Tab</h1> | |
| </label> | ||
| </div> | ||
|
|
||
| <div class="ttcont"> | ||
| <div class="texts"> | ||
| <div class="bigText" id="weatherProviderText">Weather Provider</div> | ||
| <div class="infoText" id="weatherProviderSubtext">Select the weather | ||
| service</div> | ||
| </div> | ||
| <select id="weatherProviderSelect" class="dropdown" | ||
| style="background: var(--surface); color: var(--on-surface); border: 1px solid var(--outline); border-radius: 8px; padding: 4px 8px;"> | ||
| <option value="weatherapi">WeatherAPI</option> | ||
| <option value="openmeteo">Open-Meteo</option> | ||
| </select> | ||
| </div> | ||
|
|
||
| <div class="ttcont unflex" id="locationCont"> | ||
| <div class="texts"> | ||
| <div class="bigText" id="UserLocText">Enter your Location</div> | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -192,20 +192,43 @@ <h3>External APIs</h3> | |
| which sites the extension can access from your browser settings.</p> | ||
| </li> | ||
|
|
||
| <li><strong>Location Detection (<a href="https://ipinfo.io/" target="_blank">IPinfo</a> / <a | ||
| href="https://geojs.io/" target="_blank">GeoJS</a>):</strong> | ||
| <ul> | ||
| <li><code>https://ipinfo.io/json</code> – fetches approximate city-level location based on your IP | ||
| for local weather display.</li> | ||
| <li><code>https://get.geojs.io/v1/ip/geo.json</code> – alternative used for IP-based geolocation | ||
| when Open-Meteo is active.</li> | ||
| </ul> | ||
| </li> | ||
|
|
||
| <li><strong>Weather Updates:</strong> | ||
| <ul> | ||
| <li><a href="https://www.weatherapi.com/" target="_blank">WeatherAPI</a> | ||
| <ul> | ||
| <li><code>https://api.weatherapi.com/v1/forecast.json?key=*</code> – retrieves daily | ||
| forecasts based | ||
| on your location and language preferences.</li> | ||
| <li><code>https://api.weatherapi.com/v1/search.json?key=*</code> – used to fetch location | ||
| suggestions when searching for a city.</li> | ||
| <li><code>https://cdn.weatherapi.com/weather/128x128/</code> – retrieves weather condition | ||
| icons.</li> | ||
| </ul> | ||
| </li> | ||
| <li><a href="https://open-meteo.com/" target="_blank">Open-Meteo</a> | ||
| <ul> | ||
| <li><code>https://api.open-meteo.com/v1/forecast</code> – alternative provider for | ||
| retrieving forecasts.</li> | ||
| <li><code>https://geocoding-api.open-meteo.com/v1/search</code> – alternative source for | ||
| finding location coordinates.</li> | ||
| </ul> | ||
| </li> | ||
| <li><a href="https://nominatim.org/" target="_blank">Nominatim (OpenStreetMap)</a> | ||
| <ul> | ||
| <li><code>https://nominatim.openstreetmap.org/reverse</code> – converts GPS coordinates to | ||
| readable area names when Open-Meteo and Auto Location are enabled.</li> | ||
| </ul> | ||
| </li> | ||
| </ul> | ||
|
coderabbitai[bot] marked this conversation as resolved.
|
||
| </li> | ||
|
|
||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Add descriptive alt text to badge images for accessibility.
The badge images on this line are missing alt text. Each
should include descriptive text between![and]to comply with the MD045 markdown linting rule and to ensure screen readers can identify the badge purpose.Example:
[](url)🔧 Proposed fix for badge alt text
📝 Committable suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 23-23: Images should have alternate text (alt text)
(MD045, no-alt-text)
[warning] 23-23: Images should have alternate text (alt text)
(MD045, no-alt-text)
[warning] 23-23: Images should have alternate text (alt text)
(MD045, no-alt-text)
[warning] 23-23: Images should have alternate text (alt text)
(MD045, no-alt-text)
[warning] 23-23: Images should have alternate text (alt text)
(MD045, no-alt-text)
[warning] 23-23: Images should have alternate text (alt text)
(MD045, no-alt-text)
[warning] 23-23: Images should have alternate text (alt text)
(MD045, no-alt-text)
🤖 Prompt for AI Agents
Source: Linters/SAST tools