eclipse / swtchart

Eclipse Public License 2.0
44 stars 41 forks source link

Replaced the sortable checkbox with a button and icon #249

Closed Mailaender closed 3 years ago

Mailaender commented 3 years ago

Closes https://github.com/eclipse/swtchart/issues/248.

https://github.com/eclipse/webtools.sourceediting/blob/62e575d162faf85b4b63833b193451f8da5085c5/json/bundles/org.eclipse.wst.json.ui/icons/full/obj16/sort.gif is where the icon was harvested from.

eselmeister commented 3 years ago

I would recommend to use two different icons to indicate if sorting is enabled or disabled.

Mailaender commented 3 years ago

Good point. Done. https://github.com/eclipse/capella/blob/409a90c4a2ed6c9c355badee0d068c32ec47f195/core/plugins/org.polarsys.capella.core.ui.resources/icons/full/obj16/capella_sort_disabled.gif is the source for the second grey scaled icon.