Skip to content

feat: add subnet mask tooltip to CIDR notation cells#50

Open
Chris-Hailes wants to merge 1 commit into
ckabalan:mainfrom
Chris-Hailes:feature/cidr-subnet-mask-tooltip
Open

feat: add subnet mask tooltip to CIDR notation cells#50
Chris-Hailes wants to merge 1 commit into
ckabalan:mainfrom
Chris-Hailes:feature/cidr-subnet-mask-tooltip

Conversation

@Chris-Hailes

Copy link
Copy Markdown

Hovering over a subnet address in the table now shows a Bootstrap tooltip displaying the full dotted-decimal subnet mask (e.g. 255.255.255.0) derived from the CIDR prefix length.

  • Add cidr_to_mask() helper using bitwise conversion
  • Add data-bs-toggle/placement/title attributes to .row_address
  • Init tooltips after table render; dispose before re-render to prevent leaks

Hovering over a subnet address in the table now shows a Bootstrap tooltip
displaying the full dotted-decimal subnet mask (e.g. 255.255.255.0) derived
from the CIDR prefix length.

- Add cidr_to_mask() helper using bitwise conversion
- Add data-bs-toggle/placement/title attributes to .row_address <td>
- Init tooltips after table render; dispose before re-render to prevent leaks
@sharevb

sharevb commented Apr 9, 2026

Copy link
Copy Markdown

Hi @Chris-Hailes , if you are interesting to integrate this here: https://github.com/sharevb/visualsubnetcalc (fork of this repo, that integrates also in IT Tools)

@Chris-Hailes

Copy link
Copy Markdown
Author

Hi @Chris-Hailes , if you are interesting to integrate this here: https://github.com/sharevb/visualsubnetcalc (fork of this repo, that integrates also in IT Tools)

Done

@sharevb

sharevb commented Apr 19, 2026

Copy link
Copy Markdown

Hi @Chris-Hailes , sorry but I do not see a PR for this in my fork : https://github.com/sharevb/visualsubnetcalc/pulls

@Chris-Hailes

Copy link
Copy Markdown
Author

Hi @Chris-Hailes , sorry but I do not see a PR for this in my fork : https://github.com/sharevb/visualsubnetcalc/pulls

@sharevb sorry for the delay, submit the PR now my bad

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