search

brvphoenix / luci-app-wrtbwmon

Bandwidth monitor for OpenWRT
251 stars 53 forks source link

Menu: conver to client side JS. And fix a syntax error. #30

Closed kongfl888 closed 3 years ago

kongfl888 commented 4 years ago

Since the new version of Luci is used, it is not appropriate to use controller/*.lua again.

Let it all use js.

And fix a syntax error which will case cause errors in Edge browsers and some mobile browsers.

brvphoenix commented 4 years ago

The JSON menu was supported after https://github.com/openwrt/luci/commit/cde76b0295bf128fa1117a1a30f0eaa9551fa186 which means that only the OpenWrt 19.07.3 and newer version LuCI can use the JSON menu.

This commit will introduce some compatibility issues so that it can't been merged immediately.