Skip to content

Conversation

jcostom
Copy link

@jcostom jcostom commented Jul 10, 2018

Hey -

Seriously great tool here. I've made some improvements on it (I hope)...

  • Got rid of .gif images, replaced with .png - unencumbered image formats are better. Yes, the PNGs are slightly larger, but it's not absurd.
  • Pulled form out of tables and restyled it using div's.
  • Got rid of top table as well, as it really wasn't needed.
  • Lots of stylistic improvements - introduced Google fonts use, so can be consistently displayed even on systems without Arial & Verdana. Most text in a proportional font (Lato), IP addr's in a monospaced font (Source Code Pro), Green & Red buttons

I hope you're willing to accept my proposals! Thanks for creating a nice tool.

jcostom added 6 commits July 6, 2018 13:28
* Change from GIF to PNG (unencumbered image formats are better)
* Modularized - Moved CSS and JS to their own files
* Got rid of table at top of page, changed for <div>
* Intro Google Fonts
	* Lato for most
	* Source Code Pro for addresses
* Overall beautify
Signed-off-by: Jason Costomiris <[email protected]>
@samuelleung83
Copy link

Been loving this tool of late. Would be awesome if the output could be exported to excel/spreadsheet format for further editing. But other than that, this tool has helped me immensely.

@samuelleung83
Copy link

The Show columns tickboxes appear to have stopped working. Unticking the boxes no longer hides the columns.
image

kpieckiel and others added 2 commits December 30, 2022 19:14
Large numbers of hosts (/0 and /1 subnets) are overflowing and showing
as a negative number.  Changing 1<<(32-mask) to 2**(32-mask) causes
JavaScript to calculate the integer properly.
Change shift left operator to exponent operator to avoid overflow
@drAlberT
Copy link

drAlberT commented Jan 8, 2023

The Show columns tickboxes appear to have stopped working. Unticking the boxes no longer hides the columns. image

Fixed in #11

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.

4 participants