search

discoveryjs / discovery

A framework for rapid data (JSON) analysis, shareable serverless reports and dashboards
https://discoveryjs.github.io/discovery/
MIT License
336 stars 6 forks source link

disable user-select for content-badges #15

Closed silentroach closed 5 years ago

silentroach commented 5 years ago

Привет и спасибо! :)

Проблема, с которой я периодически сталкиваюсь - я открываю json-discovery и жмакаю cmd+a и cmd+c и ожидаю, что я скопировал json целиком. А оказывается, не только его, а еще и беджики справа сверху.

Мне кажется, можно (наверное даже нужно) сделать их невыделяемыми. Что думаешь? Переключаться в Raw только для этого - ну фиг знает.

Или лучше это вообще сделать в самом расширении? Но тогда нужно будет завязываться на название класса, тоже наверное не очень хорошо <_<

lahmatiy commented 5 years ago

Да, сделать можно. Но проблема, что из struct копируется не валидный JSON – так что это не решит проблему. Кажется по этому и не стали отключать выделение на беджах. Согласен, что по cmd+a cmd+c должен копироваться оригинальный JSON. Надо подумать над решением. //сс @exdis

silentroach commented 5 years ago

json5 - тоже json :)

если ты про кавычки в ключах

или про попапы? там тоже наверное стоит убрать возможность селекта

lahmatiy commented 5 years ago

Не только, там еще сокращения и различные декоративные штуки. Думаю решить проблему можно, но решение не самое простое может быть. Пока думаю, что при выделении cmd+a – технически выделять спец область, в которой нормальный json

silentroach commented 5 years ago

Выделять мышкой-то тоже было бы неплохо, не только по cmd+a :}

exdis commented 5 years ago

В JsonDiscovery есть спецальный режим Raw, в котором как раз можно через ctrl/cmd+a выделить и скопировать сырой JSON, также для удобства есть кнопка "Copy raw". Однако, можно рассмотреть возможность в других режимах через ctrl/cmd+a выделять/копировать сырой JSON, либо же кнопку "Copy raw" иметь на всех режимах. Только вот не уверен по части UX в таких решениях.