Skip to content

feat(ui): match avatar color other platforms#933

Merged
danditomaso merged 3 commits into
meshtastic:mainfrom
danditomaso:feat(ui)--match-avatar-color-to-other-platforms
Nov 6, 2025
Merged

feat(ui): match avatar color other platforms#933
danditomaso merged 3 commits into
meshtastic:mainfrom
danditomaso:feat(ui)--match-avatar-color-to-other-platforms

Conversation

@danditomaso

Copy link
Copy Markdown
Collaborator

Description

This pull request refactors how avatar colors and labels are determined throughout the web app. Instead of generating avatar colors from user names or text, avatars now use the node number (nodeNum) to derive a unique and consistent color. The Avatar component and its usages have been updated to accept a nodeNum prop, and a new utility function, getColorFromNodeNum, has been introduced to extract RGB color values directly from the node number. Comprehensive tests for color utilities have also been added.

Avatar and color handling improvements:

  • The Avatar component now takes a nodeNum prop instead of text, using the node number to determine both the displayed initials and the avatar color. This ensures that avatar colors are consistent and unique per node. (packages/web/src/components/UI/Avatar.tsx, [[1]](https://github.com/meshtastic/web/pull/933/files#diff-a40e88caa2f5a4effe352600cb9317b758e6ff82c2fdb8bac365929d637d796bL14-R49), [[2]](https://github.com/meshtastic/web/pull/933/files#diff-a40e88caa2f5a4effe352600cb9317b758e6ff82c2fdb8bac365929d637d796bL1-R2), [[3]](https://github.com/meshtastic/web/pull/933/files#diff-a40e88caa2f5a4effe352600cb9317b758e6ff82c2fdb8bac365929d637d796bL14-R49))
  • All usages of the Avatar component across the app have been updated to pass nodeNum rather than a text label. ([[1]](https://github.com/meshtastic/web/pull/933/files#diff-698259439e8387b7bbd45e2c6e60131a824560730cc825ac462e6afc7833c16eL129-R129), [[2]](https://github.com/meshtastic/web/pull/933/files#diff-4ad8757fe14e16213f65ba23a00db65192fb49e371a5713420fcfb93048b2aeaL71-R71), [[3]](https://github.com/meshtastic/web/pull/933/files#diff-e589dcdaae1294369fc74a987a4b8ad742c54ac24eff81d4b581d70746f9d259L55-R55), [[4]](https://github.com/meshtastic/web/pull/933/files#diff-3d92f4d8e95225c51475730e639afa42f106005447b13e30652c134b2a43e4a2L208-R209), [[5]](https://github.com/meshtastic/web/pull/933/files#diff-edd299b07c7a7ca7ecf9942495cdc68c607fd3a40cac408244cf069307f7ed5fL286-R286), [[6]](https://github.com/meshtastic/web/pull/933/files#diff-4b40014725117c382ad3903cc1e4f4cb3134021b7d2ecde53f9f51ca7ccc3c0dL155-R155), [[7]](https://github.com/meshtastic/web/pull/933/files#diff-3ada7788f84cafa3153e2d91521817e312703fd025bd618ee6a1b127817a4dffL147-R145))

Color utility refactor and tests:

  • Introduced getColorFromNodeNum in color.ts, which generates an RGB color directly from the node number, and removed the previous getColorFromText function. The RGBColor type was simplified by removing the unused alpha channel. ([packages/web/src/core/utils/color.tsL5-R25](https://github.com/meshtastic/web/pull/933/files#diff-ab14bae643bc45732b144a47e8a4e6d00416e8ef6669e9de32c904d960d8c2dcL5-R25))
  • Added extensive unit tests for color utilities, including getColorFromNodeNum, hexToRgb, rgbToHex, and isLightColor. ([packages/web/src/core/utils/color.test.tsR1-R93](https://github.com/meshtastic/web/pull/933/files#diff-8c8136cc68cdfbaf15e5b05695e20b292ba2a35a85c80d48040db88193784f19R1-R93))

Component and prop updates:

  • Updated component props and type definitions to pass and use nodeNum and user protobufs instead of custom user objects with shortName and longName. This affects DeviceInfoPanel, Sidebar, and related components. ([[1]](https://github.com/meshtastic/web/pull/933/files#diff-3ada7788f84cafa3153e2d91521817e312703fd025bd618ee6a1b127817a4dffL28-R29), [[2]](https://github.com/meshtastic/web/pull/933/files#diff-3ada7788f84cafa3153e2d91521817e312703fd025bd618ee6a1b127817a4dffR3), [[3]](https://github.com/meshtastic/web/pull/933/files#diff-9f7203cea7dfe05bb4739f18e3e464760a91e8ecb417de4eb78642190d3ed212L207-R207))
  • Updated logic in components like MessageItem to derive display names and node numbers appropriately for avatar rendering. ([[1]](https://github.com/meshtastic/web/pull/933/files#diff-3d92f4d8e95225c51475730e639afa42f106005447b13e30652c134b2a43e4a2L147-R147), [[2]](https://github.com/meshtastic/web/pull/933/files#diff-3d92f4d8e95225c51475730e639afa42f106005447b13e30652c134b2a43e4a2R160))

Map and visualization consistency:

  • Map layers and markers now use getColorFromNodeNum for consistent color assignment based on node number, improving visual consistency across the application. ([[1]](https://github.com/meshtastic/web/pull/933/files#diff-c577749493ffedbb9365e011c0da6d97bb4dfb5d86489d8447ce5650d28652ceL1-L4), [[2]](https://github.com/meshtastic/web/pull/933/files#diff-c577749493ffedbb9365e011c0da6d97bb4dfb5d86489d8447ce5650d28652ceL49-R48))

These changes make avatar colors more predictable and unique per node, simplify the color logic, and improve maintainability and test coverage of the color-related utilities.

Screenshots (if applicable)

Checklist

  • Code follows project style guidelines
  • Documentation has been updated or added
  • Tests have been added or updated
  • All i18n translation labels have been added (read
    CONTRIBUTING_I18N_DEVELOPER_GUIDE.md for more details)

Copilot AI review requested due to automatic review settings November 5, 2025 18:34
@vercel

vercel Bot commented Nov 5, 2025

Copy link
Copy Markdown

@danditomaso is attempting to deploy a commit to the Meshtastic Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the Avatar component to use nodeNum directly instead of text strings, enabling consistent color generation based on node numbers rather than text hashing. This ensures all avatars for the same node display with the same color across the application.

Key changes:

  • Replaced hash-based color generation from text with direct RGB extraction from node numbers
  • Updated Avatar component to accept nodeNum instead of text and internally fetch node data
  • Removed the alpha channel from RGBColor interface and related functions
  • Added comprehensive test coverage for color utility functions

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/web/src/core/utils/color.ts Replaced getColorFromText with getColorFromNodeNum to extract RGB directly from node numbers; removed alpha channel from RGBColor interface
packages/web/src/core/utils/color.test.ts Added comprehensive test coverage for all color utility functions
packages/web/src/components/UI/Avatar.tsx Updated to accept nodeNum, fetch node data internally, and use new color generation function
packages/web/src/pages/Nodes/index.tsx Changed Avatar prop from text={shortName} to nodeNum={node.num}
packages/web/src/pages/Messages.tsx Changed Avatar prop from text={node.user?.shortName ?? t("unknown.shortName")} to nodeNum={node.num}
packages/web/src/components/PageComponents/Messages/MessageItem.tsx Changed Avatar prop from text={shortName} to nodeNum={nodeNum}
packages/web/src/components/PageComponents/Map/Popups/NodeDetail.tsx Changed Avatar prop from text={shortName} to nodeNum={node.num}
packages/web/src/components/PageComponents/Map/Markers/NodeMarker.tsx Changed Avatar prop from text={label} to nodeNum={id}
packages/web/src/components/PageComponents/Map/Layers/PrecisionLayer.tsx Replaced text-based color generation with getColorFromNodeNum(node.num)
packages/web/src/components/DeviceInfoPanel.tsx Changed Avatar prop from text={user.shortName} to nodeNum={Number(user.id)} and updated user prop type
packages/web/src/components/Sidebar.tsx Simplified user prop to pass myNode.user directly instead of object with extracted properties
packages/web/src/components/CommandPalette/index.tsx Simplified Avatar usage to only pass nodeNum

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/web/src/components/UI/Avatar.tsx Outdated
Comment thread packages/web/src/components/DeviceInfoPanel.tsx Outdated
danditomaso and others added 2 commits November 5, 2025 16:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@vercel

vercel Bot commented Nov 5, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
web-test Ready Ready Preview Comment Nov 5, 2025 10:02pm

@danditomaso danditomaso merged commit a00cb20 into meshtastic:main Nov 6, 2025
4 checks passed
@danditomaso danditomaso deleted the feat(ui)--match-avatar-color-to-other-platforms branch November 6, 2025 02:53
danditomaso added a commit that referenced this pull request Feb 10, 2026
* Add badge indicator on layers icon. (#894)

* feat: indicate number of layers enabled.

* Update packages/web/src/components/PageComponents/Map/Tools/MapLayerTool.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: reduced code duplication in layers component

* fixed unread message bubble

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(actions): improve main to stable release workflow (#895)

* fix(actions): improve main to stable release workflow

* Update .github/workflows/update-stable-from-master.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update .github/workflows/update-stable-from-master.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(ui): logic on waypoint layer component caused 0 to be shown in UI (#896)

* fix(ui): logic on waypoint layer component caused 0 to be shown in UI

* Update packages/web/src/components/PageComponents/Map/Popups/WaypointDetail.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#899)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* Update readme with new widgets (#901)

* chore: add new widgets to readme

* add docs url

* Update README.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* feat(i18n): add fr localization support (#902)

* feat: add fr support

* fix(i18n): ensure langs are sorted before being displayed.

* feat(ci): add CI workflow to automatically close issue after 60 days (#897)

* feat(ci): add CI workflow to automatically close issue after 60 days

* run once pe github issue to run once per day

* chore(deps): bump vite from 7.1.9 to 7.1.11 (#903)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 7.1.9 to 7.1.11.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v7.1.11/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-version: 7.1.11
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update inactive issue workflow schedule and settings (#905)

* Update inactive issue workflow schedule and settings

* Update .github/workflows/inactive-issue.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Persists device and app stores across sessions (#860)

* Persistence for device and app data

* esphemeral -> ephemeral

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* devices -> app

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Additional waypoint methods, update mock, update tests

---------

Co-authored-by: philon- <philon-@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix description for historyReturnWindow (#907)

It's not number of records but the time window...

* Feat(config): Align settings menu to match android/ios (#906)

* feat: aligned settings menu to match android/ios

* updated sidebar text size.

* Update packages/web/public/i18n/locales/en/config.json

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/public/i18n/locales/en/config.json

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/Settings/User.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/ModuleConfig/Telemetry.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* linting/formatting fixes

* fixed formatting issue

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore: add new folders to biome config (#910)

* chore(i18n): New Crowdin Translations by GitHub Action (#908)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix: use correct deprecated GPS coordinate format enum (#917)

The Config_DisplayConfig_GpsCoordinateFormat export doesn't exist in the protobufs package. The correct export is Config_DisplayConfig_DeprecatedGpsCoordinateFormat, which matches what's used in the validation schema.

Added TODO comment explaining that this field is deprecated since protobufs 2.7.4 and should be migrated to DeviceUIConfig.gps_format when DeviceUI settings are implemented.

* style: fix line wrapping for GPS coordinate format enum (#918)

- Split long enum reference across multiple lines to improve code readability
- Maintains consistent code formatting standards without changing functionality

* feat(state): enable deviceStore persistance (#922)

* fix(core): ensure core package works in browser (#923)

* fix(core): ensure core package works in browser

* style(core): revert new line removal

* fix: add @serialport/bindings-cpp to onlyBuiltDependencies (#914)

* feat(ui): Add UI library (#900)

* feat: scaffold UI library

* Update packages/ui/src/components/theme-provider.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* add lock file

* lint/formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: added skelton loader for message items (#927)

* feat(connections): Add connections page (replaces new device dialog) (#919)

* feat(conn): add connection screen and logic

* fixes from code review

* force https

* code review fixes

* add http for self testing

* enable deviceStore persistance

* added translations

* disabled feature flag

* i18n updates

* chore: add new folders to biome config (#910)

* chore(i18n): New Crowdin Translations by GitHub Action (#908)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix: use correct deprecated GPS coordinate format enum (#917)

The Config_DisplayConfig_GpsCoordinateFormat export doesn't exist in the protobufs package. The correct export is Config_DisplayConfig_DeprecatedGpsCoordinateFormat, which matches what's used in the validation schema.

Added TODO comment explaining that this field is deprecated since protobufs 2.7.4 and should be migrated to DeviceUIConfig.gps_format when DeviceUI settings are implemented.

* style: fix line wrapping for GPS coordinate format enum (#918)

- Split long enum reference across multiple lines to improve code readability
- Maintains consistent code formatting standards without changing functionality

* fix(core): ensure core package works in browser (#923)

* fix(core): ensure core package works in browser

* style(core): revert new line removal

* fix: add @serialport/bindings-cpp to onlyBuiltDependencies (#914)

* feat(ui): Add UI library (#900)

* feat: scaffold UI library

* Update packages/ui/src/components/theme-provider.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* add lock file

* lint/formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* formatting/linting fixes

* fixed some paring logic

* fixed connection issue with serial

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Crowdin Bot <support+bot@crowdin.com>
Co-authored-by: Kamil Dzieniszewski <kamil.dzieniszewski@gmail.com>
Co-authored-by: Azarattum <43073346+Azarattum@users.noreply.github.com>
Co-authored-by: Ben Allfree <ben@benallfree.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(ci): add ui library to excluded list (#928)

* fix(config): update change registry channel value (#929)

* fix(config): update change registry channel value

* format/linting

* chore(i18n): New Crowdin Translations by GitHub Action (#924)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix(connections): ensure connections reflect actual status. (#930)

* fix(connection): support port on HTTP connection (#935)

* feat(docker): add arm v7 support (#934)

* feat(ui): match avatar color other platforms (#933)

* feat(ui): match avatar color other platforms

* Update packages/web/src/components/UI/Avatar.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/DeviceInfoPanel.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(ui): fix add connection dialog typo (#938)

* fix(i18n): Correct 'disconnected' typo in connections.json (#943)

* fix(i18n): Correct 'disconnected' typo in connections.json

* fix(i18n): Correct typo 'checkConnetion' to 'checkConnection'

* Fix(i18n): Correct typo in connection test description

---------

Co-authored-by: philon- <philon-@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#941)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* Add 'packages/web' to excluded directories (#947)

* refactor: device connection logic, added nonce to get config only (#946)

* refactor: device connection logic, added nonce to get config only on connect.

* Update packages/web/src/core/services/MeshService.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/pages/Connections/useConnections.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* code review fixes

* fixes from code review

* ui fixes

* refactored meshService, moved code into deviceStore. Fixed some connnection issues

* formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: removed duplicate images (#951)

* feat: add devcontainer (#953)

* fix(ui): add language switcher to connections page (#954)

* fix(ui): add language switcher to connections page

* desciption length fix

* Update packages/web/src/pages/Connections/index.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* add new language picker key

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#958)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix: interpolate longName and shortName in PKI backup download (#959)

This caused {{shortName}} and {{longName}} to appear unformatted in the
exported key files:

``` === MESHTASTIC KEYS FOR {{longName}} ({{shortName}}) ===

Private Key: <censored>

Public Key: <censored>

=== END OF KEYS === ```

The fix simply replicates the behaviour used elsewhere in
PKIIBackupDialog.

* fix(ui): removed internet hosted fonts from app (#955)

* fix(ui): removed internet hosted fonts from app

* Update packages/web/src/components/generic/Mono.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/UI/Typography/Code.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* removed unsupported font extention

* formatter fix

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#962)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix(ui): correct typo in languagePicker key and adjust description width (#961)

Fixed typo 'languagePickeer' to 'languagePicker' in DeviceInfoPanel and corrected responsive width class from 'md:w-6' to 'md:w-5/6' in Connections page description.

* chore(deps-dev): bump happy-dom from 20.0.0 to 20.0.2 (#968)

Bumps [happy-dom](https://github.com/capricorn86/happy-dom) from 20.0.0 to 20.0.2.
- [Release notes](https://github.com/capricorn86/happy-dom/releases)
- [Commits](capricorn86/happy-dom@v20.0.0...v20.0.2)

---
updated-dependencies:
- dependency-name: happy-dom
  dependency-version: 20.0.2
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(ui): add "never" i18n string, fix "Favorite" tooltip (#965)

* fix(ui): add "never" i18n string, fix "Favorite" tooltip

* format

* format

---------

Co-authored-by: Pmmlabs <meshtastic@pmmlabs.ru>

* feat: add fixed position coordinate picker  (#909)

* chore: remove unused logo SVG files

* feat: add interactive fixed position picker with map interface

- Created new FixedPositionPicker component with clickable map for setting device coordinates
- Added form field type for fixed position picker that appears when fixedPosition toggle is enabled
- Implemented position request functionality to retrieve current device location

* feat: display altitude unit based on user's display settings

- Added dynamic altitude unit (Meters/Feet) that respects the user's imperial/metric display preference
- Updated altitude field description to show the appropriate unit instead of hardcoded "Meters"

* refactor: replace any type with MapLayerMouseEvent in map click handler

* refactor: improve accessibility and code quality in FixedPositionPicker

- Replace hardcoded IDs with useId() hook for proper accessibility
- Use Number.isNaN() instead of isNaN() for more reliable type checking
- Add radix parameter to parseInt() and remove unnecessary fragment wrapper

* refactor: simplify fixed position picker integration

- Removed dedicated FixedPositionPicker form field type in favor of toggle's additionalContent prop
- Moved FixedPositionPicker to render conditionally within toggle field instead of as separate dynamic field
- Streamlined form field types by eliminating FixedPositionPickerFieldProps

* style: format code with consistent line breaks and import ordering

* refactor: simplify fixed position picker container styling

* feat: disable fixed position toggle when GPS is enabled

* refactor: use ComponentRef instead of ElementRef in Switch component

* refactor: replace interactive map picker with inline coordinate fields for fixed position

- Removed FixedPositionPicker component with map interface
- Added latitude, longitude, and altitude fields directly to position form
- Moved coordinate validation into PositionValidationSchema with proper min/max bounds
- Updated translation strings to include coordinate ranges and improved altitude description
- Coordinates now sent via setFixedPosition admin message on form submit when fixedPosition is enabled

* refactor: simplify toggle field by removing additionalContent prop and unused field spreading

- Removed additionalContent prop and its JSDoc documentation from ToggleFieldProps
- Removed rendering of additionalContent below toggle switch
- Cleaned up Controller render function by removing unused rest spread operator
- Renamed field destructuring to controllerField for clarity

* refactor: improve fixed position handling and add position broadcast request

- Restructure onSubmit to save config before sending admin message
- Add position broadcast request after setting fixed position to immediately update display
- Add comprehensive debug logging throughout submission flow
- Extract coordinate exclusion logic earlier in submission process for clarity
- Add 1 second delay before requesting position broadcast to allow fixed position processing

* feat: add max length constraint to latitude and longitude fields

- Set fieldLength.max to 10 for both latitude and longitude inputs
- Prevents excessive decimal precision while maintaining 7 decimal places (±1.1cm accuracy)

* feat(ui): add SNR, RSSI, hops info for messages (#963)

* feat(ui): add SNR, RSSI, hops, MQTT info for messages

* review fixes

* zeros for new fields

* Move label under the message

---------

Co-authored-by: Pmmlabs <meshtastic@pmmlabs.ru>

* feat(map): add heatmap layer (#969)

* feat: add heatmap layer

* Update packages/web/src/components/PageComponents/Map/Layers/HeatmapLayer.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/Map/Tools/MapLayerTool.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/pages/Map/index.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/pages/Map/index.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/Map/Layers/HeatmapLayer.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* lint/formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Revert "feat(ui): add SNR, RSSI, hops info for messages (#963)" (#974)

This reverts commit 020e9d6.

* Update README with Buf CLI installation instructions (#981)

Added instructions for installing the Buf CLI.

* Channel bandwidth is kHz, not MHz (#983)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Crowdin Bot <support+bot@crowdin.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jeremy Gallant <8975765+philon-@users.noreply.github.com>
Co-authored-by: philon- <philon-@users.noreply.github.com>
Co-authored-by: Wessel <wessel@weebl.me>
Co-authored-by: Kamil Dzieniszewski <kamil.dzieniszewski@gmail.com>
Co-authored-by: Azarattum <43073346+Azarattum@users.noreply.github.com>
Co-authored-by: Ben Allfree <ben@benallfree.com>
Co-authored-by: jsacrist <jsacrist@users.noreply.github.com>
Co-authored-by: zeo <goonox35@proton.me>
Co-authored-by: Kamil Dzieniszewski <kamil.d@playsmart.io>
Co-authored-by: Alexey Stepanov <Pmmlabs@users.noreply.github.com>
Co-authored-by: Pmmlabs <meshtastic@pmmlabs.ru>
Co-authored-by: rj-xy <2442596+rj-xy@users.noreply.github.com>
danditomaso added a commit that referenced this pull request Feb 11, 2026
* Add badge indicator on layers icon. (#894)

* feat: indicate number of layers enabled.

* Update packages/web/src/components/PageComponents/Map/Tools/MapLayerTool.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: reduced code duplication in layers component

* fixed unread message bubble

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(actions): improve main to stable release workflow (#895)

* fix(actions): improve main to stable release workflow

* Update .github/workflows/update-stable-from-master.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update .github/workflows/update-stable-from-master.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(ui): logic on waypoint layer component caused 0 to be shown in UI (#896)

* fix(ui): logic on waypoint layer component caused 0 to be shown in UI

* Update packages/web/src/components/PageComponents/Map/Popups/WaypointDetail.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#899)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* Update readme with new widgets (#901)

* chore: add new widgets to readme

* add docs url

* Update README.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* feat(i18n): add fr localization support (#902)

* feat: add fr support

* fix(i18n): ensure langs are sorted before being displayed.

* feat(ci): add CI workflow to automatically close issue after 60 days (#897)

* feat(ci): add CI workflow to automatically close issue after 60 days

* run once pe github issue to run once per day

* chore(deps): bump vite from 7.1.9 to 7.1.11 (#903)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 7.1.9 to 7.1.11.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v7.1.11/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-version: 7.1.11
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update inactive issue workflow schedule and settings (#905)

* Update inactive issue workflow schedule and settings

* Update .github/workflows/inactive-issue.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Persists device and app stores across sessions (#860)

* Persistence for device and app data

* esphemeral -> ephemeral

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* devices -> app

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Additional waypoint methods, update mock, update tests

---------

Co-authored-by: philon- <philon-@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix description for historyReturnWindow (#907)

It's not number of records but the time window...

* Feat(config): Align settings menu to match android/ios (#906)

* feat: aligned settings menu to match android/ios

* updated sidebar text size.

* Update packages/web/public/i18n/locales/en/config.json

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/public/i18n/locales/en/config.json

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/Settings/User.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/ModuleConfig/Telemetry.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* linting/formatting fixes

* fixed formatting issue

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore: add new folders to biome config (#910)

* chore(i18n): New Crowdin Translations by GitHub Action (#908)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix: use correct deprecated GPS coordinate format enum (#917)

The Config_DisplayConfig_GpsCoordinateFormat export doesn't exist in the protobufs package. The correct export is Config_DisplayConfig_DeprecatedGpsCoordinateFormat, which matches what's used in the validation schema.

Added TODO comment explaining that this field is deprecated since protobufs 2.7.4 and should be migrated to DeviceUIConfig.gps_format when DeviceUI settings are implemented.

* style: fix line wrapping for GPS coordinate format enum (#918)

- Split long enum reference across multiple lines to improve code readability
- Maintains consistent code formatting standards without changing functionality

* feat(state): enable deviceStore persistance (#922)

* fix(core): ensure core package works in browser (#923)

* fix(core): ensure core package works in browser

* style(core): revert new line removal

* fix: add @serialport/bindings-cpp to onlyBuiltDependencies (#914)

* feat(ui): Add UI library (#900)

* feat: scaffold UI library

* Update packages/ui/src/components/theme-provider.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* add lock file

* lint/formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: added skelton loader for message items (#927)

* feat(connections): Add connections page (replaces new device dialog) (#919)

* feat(conn): add connection screen and logic

* fixes from code review

* force https

* code review fixes

* add http for self testing

* enable deviceStore persistance

* added translations

* disabled feature flag

* i18n updates

* chore: add new folders to biome config (#910)

* chore(i18n): New Crowdin Translations by GitHub Action (#908)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix: use correct deprecated GPS coordinate format enum (#917)

The Config_DisplayConfig_GpsCoordinateFormat export doesn't exist in the protobufs package. The correct export is Config_DisplayConfig_DeprecatedGpsCoordinateFormat, which matches what's used in the validation schema.

Added TODO comment explaining that this field is deprecated since protobufs 2.7.4 and should be migrated to DeviceUIConfig.gps_format when DeviceUI settings are implemented.

* style: fix line wrapping for GPS coordinate format enum (#918)

- Split long enum reference across multiple lines to improve code readability
- Maintains consistent code formatting standards without changing functionality

* fix(core): ensure core package works in browser (#923)

* fix(core): ensure core package works in browser

* style(core): revert new line removal

* fix: add @serialport/bindings-cpp to onlyBuiltDependencies (#914)

* feat(ui): Add UI library (#900)

* feat: scaffold UI library

* Update packages/ui/src/components/theme-provider.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* add lock file

* lint/formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* formatting/linting fixes

* fixed some paring logic

* fixed connection issue with serial

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Crowdin Bot <support+bot@crowdin.com>
Co-authored-by: Kamil Dzieniszewski <kamil.dzieniszewski@gmail.com>
Co-authored-by: Azarattum <43073346+Azarattum@users.noreply.github.com>
Co-authored-by: Ben Allfree <ben@benallfree.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(ci): add ui library to excluded list (#928)

* fix(config): update change registry channel value (#929)

* fix(config): update change registry channel value

* format/linting

* chore(i18n): New Crowdin Translations by GitHub Action (#924)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix(connections): ensure connections reflect actual status. (#930)

* fix(connection): support port on HTTP connection (#935)

* feat(docker): add arm v7 support (#934)

* feat(ui): match avatar color other platforms (#933)

* feat(ui): match avatar color other platforms

* Update packages/web/src/components/UI/Avatar.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/DeviceInfoPanel.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(ui): fix add connection dialog typo (#938)

* fix(i18n): Correct 'disconnected' typo in connections.json (#943)

* fix(i18n): Correct 'disconnected' typo in connections.json

* fix(i18n): Correct typo 'checkConnetion' to 'checkConnection'

* Fix(i18n): Correct typo in connection test description

---------

Co-authored-by: philon- <philon-@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#941)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* Add 'packages/web' to excluded directories (#947)

* refactor: device connection logic, added nonce to get config only (#946)

* refactor: device connection logic, added nonce to get config only on connect.

* Update packages/web/src/core/services/MeshService.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/pages/Connections/useConnections.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* code review fixes

* fixes from code review

* ui fixes

* refactored meshService, moved code into deviceStore. Fixed some connnection issues

* formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: removed duplicate images (#951)

* feat: add devcontainer (#953)

* fix(ui): add language switcher to connections page (#954)

* fix(ui): add language switcher to connections page

* desciption length fix

* Update packages/web/src/pages/Connections/index.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* add new language picker key

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#958)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix: interpolate longName and shortName in PKI backup download (#959)

This caused {{shortName}} and {{longName}} to appear unformatted in the
exported key files:

``` === MESHTASTIC KEYS FOR {{longName}} ({{shortName}}) ===

Private Key: <censored>

Public Key: <censored>

=== END OF KEYS === ```

The fix simply replicates the behaviour used elsewhere in
PKIIBackupDialog.

* fix(ui): removed internet hosted fonts from app (#955)

* fix(ui): removed internet hosted fonts from app

* Update packages/web/src/components/generic/Mono.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/UI/Typography/Code.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* removed unsupported font extention

* formatter fix

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* chore(i18n): New Crowdin Translations by GitHub Action (#962)

Co-authored-by: Crowdin Bot <support+bot@crowdin.com>

* fix(ui): correct typo in languagePicker key and adjust description width (#961)

Fixed typo 'languagePickeer' to 'languagePicker' in DeviceInfoPanel and corrected responsive width class from 'md:w-6' to 'md:w-5/6' in Connections page description.

* chore(deps-dev): bump happy-dom from 20.0.0 to 20.0.2 (#968)

Bumps [happy-dom](https://github.com/capricorn86/happy-dom) from 20.0.0 to 20.0.2.
- [Release notes](https://github.com/capricorn86/happy-dom/releases)
- [Commits](capricorn86/happy-dom@v20.0.0...v20.0.2)

---
updated-dependencies:
- dependency-name: happy-dom
  dependency-version: 20.0.2
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(ui): add "never" i18n string, fix "Favorite" tooltip (#965)

* fix(ui): add "never" i18n string, fix "Favorite" tooltip

* format

* format

---------

Co-authored-by: Pmmlabs <meshtastic@pmmlabs.ru>

* feat: add fixed position coordinate picker  (#909)

* chore: remove unused logo SVG files

* feat: add interactive fixed position picker with map interface

- Created new FixedPositionPicker component with clickable map for setting device coordinates
- Added form field type for fixed position picker that appears when fixedPosition toggle is enabled
- Implemented position request functionality to retrieve current device location

* feat: display altitude unit based on user's display settings

- Added dynamic altitude unit (Meters/Feet) that respects the user's imperial/metric display preference
- Updated altitude field description to show the appropriate unit instead of hardcoded "Meters"

* refactor: replace any type with MapLayerMouseEvent in map click handler

* refactor: improve accessibility and code quality in FixedPositionPicker

- Replace hardcoded IDs with useId() hook for proper accessibility
- Use Number.isNaN() instead of isNaN() for more reliable type checking
- Add radix parameter to parseInt() and remove unnecessary fragment wrapper

* refactor: simplify fixed position picker integration

- Removed dedicated FixedPositionPicker form field type in favor of toggle's additionalContent prop
- Moved FixedPositionPicker to render conditionally within toggle field instead of as separate dynamic field
- Streamlined form field types by eliminating FixedPositionPickerFieldProps

* style: format code with consistent line breaks and import ordering

* refactor: simplify fixed position picker container styling

* feat: disable fixed position toggle when GPS is enabled

* refactor: use ComponentRef instead of ElementRef in Switch component

* refactor: replace interactive map picker with inline coordinate fields for fixed position

- Removed FixedPositionPicker component with map interface
- Added latitude, longitude, and altitude fields directly to position form
- Moved coordinate validation into PositionValidationSchema with proper min/max bounds
- Updated translation strings to include coordinate ranges and improved altitude description
- Coordinates now sent via setFixedPosition admin message on form submit when fixedPosition is enabled

* refactor: simplify toggle field by removing additionalContent prop and unused field spreading

- Removed additionalContent prop and its JSDoc documentation from ToggleFieldProps
- Removed rendering of additionalContent below toggle switch
- Cleaned up Controller render function by removing unused rest spread operator
- Renamed field destructuring to controllerField for clarity

* refactor: improve fixed position handling and add position broadcast request

- Restructure onSubmit to save config before sending admin message
- Add position broadcast request after setting fixed position to immediately update display
- Add comprehensive debug logging throughout submission flow
- Extract coordinate exclusion logic earlier in submission process for clarity
- Add 1 second delay before requesting position broadcast to allow fixed position processing

* feat: add max length constraint to latitude and longitude fields

- Set fieldLength.max to 10 for both latitude and longitude inputs
- Prevents excessive decimal precision while maintaining 7 decimal places (±1.1cm accuracy)

* feat(ui): add SNR, RSSI, hops info for messages (#963)

* feat(ui): add SNR, RSSI, hops, MQTT info for messages

* review fixes

* zeros for new fields

* Move label under the message

---------

Co-authored-by: Pmmlabs <meshtastic@pmmlabs.ru>

* feat(map): add heatmap layer (#969)

* feat: add heatmap layer

* Update packages/web/src/components/PageComponents/Map/Layers/HeatmapLayer.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/Map/Tools/MapLayerTool.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/pages/Map/index.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/pages/Map/index.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/Map/Layers/HeatmapLayer.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* lint/formatting fixes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Revert "feat(ui): add SNR, RSSI, hops info for messages (#963)" (#974)

This reverts commit 020e9d6.

* Update README with Buf CLI installation instructions (#981)

Added instructions for installing the Buf CLI.

* Channel bandwidth is kHz, not MHz (#983)

* fix channel importing bug (#1005)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Crowdin Bot <support+bot@crowdin.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jeremy Gallant <8975765+philon-@users.noreply.github.com>
Co-authored-by: philon- <philon-@users.noreply.github.com>
Co-authored-by: Wessel <wessel@weebl.me>
Co-authored-by: Kamil Dzieniszewski <kamil.dzieniszewski@gmail.com>
Co-authored-by: Azarattum <43073346+Azarattum@users.noreply.github.com>
Co-authored-by: Ben Allfree <ben@benallfree.com>
Co-authored-by: jsacrist <jsacrist@users.noreply.github.com>
Co-authored-by: zeo <goonox35@proton.me>
Co-authored-by: Kamil Dzieniszewski <kamil.d@playsmart.io>
Co-authored-by: Alexey Stepanov <Pmmlabs@users.noreply.github.com>
Co-authored-by: Pmmlabs <meshtastic@pmmlabs.ru>
Co-authored-by: rj-xy <2442596+rj-xy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants