chore: remove unused code (#187)

pskelin · web-flow · commit 7996cdb53fe2 · 2019-03-15T11:15:43.000+02:00
- the contructors with parameters are a leftover from the light weight controls
- three unused files for text align and text direction (handled on html now)
diff --git a/packages/main/src/Button.js b/packages/main/src/Button.js
@@ -161,10 +161,6 @@ class Button extends WebComponent {
 		return ButtonTemplateContext.calculate;
 	}
 
-	constructor() {
-		super();
-	}
-
 	onBeforeRendering() {
 		if (this.icon) {
 			this._iconSettings = {
diff --git a/packages/main/src/Calendar.js b/packages/main/src/Calendar.js
@@ -126,8 +126,8 @@ class Calendar extends WebComponent {
 		return CalendarRenderer;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 		this._oLocale = getFormatLocale();
 		this._oLocaleData = new LocaleData(this._oLocale);
 		this._header = {};
diff --git a/packages/main/src/CalendarHeader.js b/packages/main/src/CalendarHeader.js
@@ -58,9 +58,8 @@ class CalendarHeader extends WebComponent {
 		return CalendarHeaderRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-
+	constructor() {
+		super();
 		this._btnPrev = {};
 		this._btnPrev.icon = "sap-icon://slim-arrow-left";
 
diff --git a/packages/main/src/Card.js b/packages/main/src/Card.js
@@ -115,10 +115,6 @@ class Card extends WebComponent {
 		return CardRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	static calculateTemplateContext(state) {
 		const hasAvatar = !!state.avatar;
 		const icon = hasAvatar && isIconURI(state.avatar);
diff --git a/packages/main/src/CheckBox.js b/packages/main/src/CheckBox.js
@@ -163,8 +163,8 @@ class CheckBox extends WebComponent {
 		return CheckBoxRenderer;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 		this._label = {};
 	}
 
diff --git a/packages/main/src/CustomListItem.js b/packages/main/src/CustomListItem.js
@@ -54,10 +54,6 @@ class CustomListItem extends ListItem {
 		return CustomListItemRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	static get calculateTemplateContext() {
 		return CustomListItemTemplateContext.calculate;
 	}
diff --git a/packages/main/src/DatePicker.js b/packages/main/src/DatePicker.js
@@ -214,9 +214,8 @@ class DatePicker extends WebComponent {
 	}
 
 
-	constructor(state) {
-		super(state);
-
+	constructor() {
+		super();
 		this._input = {};
 		this._input.type = InputType.Text;
 		this._input.icon = {};
diff --git a/packages/main/src/DayPicker.js b/packages/main/src/DayPicker.js
@@ -121,8 +121,8 @@ class DayPicker extends WebComponent {
 		return DayPickerRenderer;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 		this._oLocale = getFormatLocale();
 		this._oLocaleData = new LocaleData(this._oLocale);
 
diff --git a/packages/main/src/Dialog.js b/packages/main/src/Dialog.js
@@ -81,10 +81,6 @@ class Dialog extends Popup {
 		return DialogRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	/**
 	* Opens the <code>ui5-dialog</code>.
 	* @public
diff --git a/packages/main/src/GroupHeaderListItem.js b/packages/main/src/GroupHeaderListItem.js
@@ -43,10 +43,6 @@ const metadata = {
  * @public
  */
 class GroupHeaderListItem extends ListItemBase {
-	constructor(state) {
-		super(state);
-	}
-
 	static get renderer() {
 		return GroupHeaderListItemRenderer;
 	}
diff --git a/packages/main/src/Icon.js b/packages/main/src/Icon.js
@@ -86,10 +86,6 @@ class Icon extends WebComponent {
 		return IconTemplateContext.calculate;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	focus() {
 		HTMLElement.prototype.focus.call(this);
 	}
diff --git a/packages/main/src/Input.js b/packages/main/src/Input.js
@@ -280,9 +280,8 @@ class Input extends WebComponent {
 		return InputTemplateContext.calculate;
 	}
 
-	constructor(state) {
-		super(state);
-
+	constructor() {
+		super();
 		// Indicates if there is selected suggestionItem.
 		this.hasSuggestionItemSelected = false;
 
diff --git a/packages/main/src/Label.js b/packages/main/src/Label.js
@@ -105,10 +105,6 @@ class Label extends WebComponent {
 		return metadata;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	static get renderer() {
 		return LabelRenderer;
 	}
diff --git a/packages/main/src/Link.js b/packages/main/src/Link.js
@@ -158,11 +158,6 @@ class Link extends WebComponent {
 		return LinkRederer;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
-
 	ontap(event) {
 		if (this.disabled) {
 			return;
diff --git a/packages/main/src/List.js b/packages/main/src/List.js
@@ -249,9 +249,8 @@ class List extends WebComponent {
 		return ListRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-
+	constructor() {
+		super();
 		this.initItemNavigation();
 
 		// Stores the last focused item within the internal ul element.
diff --git a/packages/main/src/ListItem.js b/packages/main/src/ListItem.js
@@ -85,9 +85,8 @@ class ListItem extends ListItemBase {
 		return metadata;
 	}
 
-	constructor(state) {
-		super(state);
-
+	constructor() {
+		super();
 		this._fnOnDelete = this.onDelete.bind(this);
 	}
 
diff --git a/packages/main/src/ListItemBase.js b/packages/main/src/ListItemBase.js
@@ -55,10 +55,6 @@ class ListItemBase extends WebComponent {
 		return metadata;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	static get calculateTemplateContext() {
 		return ListItemBaseTemplateContext.calculate;
 	}
diff --git a/packages/main/src/MonthPicker.js b/packages/main/src/MonthPicker.js
@@ -89,8 +89,8 @@ class MonthPicker extends WebComponent {
 		return MonthPickerRenderer;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 		this._oLocale = getFormatLocale();
 		this._oLocaleData = new LocaleData(this._oLocale);
 
diff --git a/packages/main/src/Panel.js b/packages/main/src/Panel.js
@@ -203,8 +203,8 @@ class Panel extends WebComponent {
 		return PanelTemplateContext.calculate;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 
 		this.resourceBundle = getResourceBundle("@ui5/webcomponents");
 
diff --git a/packages/main/src/Popover.js b/packages/main/src/Popover.js
@@ -189,8 +189,8 @@ class Popover extends Popup {
 		return PopoverRenderer;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 
 		this._documentMouseDownHandler = this.documentMouseDown.bind(this);
 
diff --git a/packages/main/src/Popup.js b/packages/main/src/Popup.js
@@ -237,8 +237,8 @@ class Popup extends WebComponent {
 		return false;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 
 		this._documentKeyDownHandler = this.documentKeyDown.bind(this);
 	}
diff --git a/packages/main/src/RadioButton.js b/packages/main/src/RadioButton.js
@@ -160,8 +160,8 @@ class RadioButton extends WebComponent {
 		return RadioButtonRenderer;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 		this._label = {};
 	}
 
diff --git a/packages/main/src/Select.js b/packages/main/src/Select.js
@@ -150,8 +150,8 @@ class Select extends WebComponent {
 		return SelectTemplateContext.calculate;
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 
 		this._setSelectedItem(null);
 		this._setPreviewedItem(null);
diff --git a/packages/main/src/ShellBar.js b/packages/main/src/ShellBar.js
@@ -287,8 +287,8 @@ class ShellBar extends WebComponent {
 		};
 	}
 
-	constructor(props) {
-		super(props);
+	constructor() {
+		super();
 
 		this._itemsInfo = [];
 		this._isInitialRendering = true;
diff --git a/packages/main/src/StandardListItem.js b/packages/main/src/StandardListItem.js
@@ -87,10 +87,6 @@ const metadata = {
  * @public
  */
 class StandardListItem extends ListItem {
-	constructor(state) {
-		super(state);
-	}
-
 	static get renderer() {
 		return StandardListItemRenderer;
 	}
diff --git a/packages/main/src/Switch.js b/packages/main/src/Switch.js
@@ -135,10 +135,6 @@ class Switch extends WebComponent {
 		return SwitchRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	ontap(event) {
 		this.toggle();
 	}
diff --git a/packages/main/src/Tab.js b/packages/main/src/Tab.js
@@ -134,10 +134,6 @@ class Tab extends TabBase {
 		return TabRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	static get calculateTemplateContext() {
 		return TabTemplateContext.calculate;
 	}
diff --git a/packages/main/src/TabBase.js b/packages/main/src/TabBase.js
@@ -24,10 +24,6 @@ class TabBase extends WebComponent {
 	static get metadata() {
 		return metadata;
 	}
-
-	constructor(state) {
-		super(state);
-	}
 }
 
 export default TabBase;
diff --git a/packages/main/src/TabContainer.js b/packages/main/src/TabContainer.js
@@ -306,8 +306,8 @@ class TabContainer extends WebComponent {
 		}
 	}
 
-	constructor(state) {
-		super(state);
+	constructor() {
+		super();
 
 		this._updateScrollingHandler = this._updateScrolling.bind(this);
 
diff --git a/packages/main/src/TabSeparator.js b/packages/main/src/TabSeparator.js
@@ -35,10 +35,6 @@ class TabSeparator extends TabBase {
 		return TabSeparatorRenderer;
 	}
 
-	constructor(state) {
-		super(state);
-	}
-
 	static get calculateTemplateContext() {
 		return TabSeparatorTemplateContext.calculate;
 	}
diff --git a/packages/main/src/Table.js b/packages/main/src/Table.js
@@ -163,8 +163,8 @@ class Table extends WebComponent {
 		};
 	}
 
-	constructor(props) {
-		super(props);
+	constructor() {
+		super();
 
 		this._itemNavigation = new ItemNavigation(this);
 
diff --git a/packages/main/src/TemplateHelper.js b/packages/main/src/TemplateHelper.js
diff --git a/packages/main/src/TextArea.js b/packages/main/src/TextArea.js
diff --git a/packages/main/src/Title.js b/packages/main/src/Title.js
diff --git a/packages/main/src/ToggleButton.js b/packages/main/src/ToggleButton.js
diff --git a/packages/main/src/YearPicker.js b/packages/main/src/YearPicker.js
diff --git a/packages/main/src/types/TextAlign.js b/packages/main/src/types/TextAlign.js
diff --git a/packages/main/src/types/TextDirection.js b/packages/main/src/types/TextDirection.js

Original file line number	Diff line number	Diff line change
`@@ -163,8 +163,8 @@ class CheckBox extends WebComponent {`
`163`	`163`	`return CheckBoxRenderer;`
`164`	`164`	`}`
`165`	`165`
`166`		`- constructor(state) {`
`167`		`- super(state);`
	`166`	`+ constructor() {`
	`167`	`+ super();`
`168`	`168`	`this._label = {};`
`169`	`169`	`}`
`170`	`170`
Original file line number	Diff line number	Diff line change
`@@ -54,10 +54,6 @@ class CustomListItem extends ListItem {`
`54`	`54`	`return CustomListItemRenderer;`
`55`	`55`	`}`
`56`	`56`
`57`		`- constructor(state) {`
`58`		`- super(state);`
`59`		`- }`
`60`		`-`
`61`	`57`	`static get calculateTemplateContext() {`
`62`	`58`	`return CustomListItemTemplateContext.calculate;`
`63`	`59`	`}`
Original file line number	Diff line number	Diff line change
`@@ -121,8 +121,8 @@ class DayPicker extends WebComponent {`
`121`	`121`	`return DayPickerRenderer;`
`122`	`122`	`}`
`123`	`123`
`124`		`- constructor(state) {`
`125`		`- super(state);`
	`124`	`+ constructor() {`
	`125`	`+ super();`
`126`	`126`	`this._oLocale = getFormatLocale();`
`127`	`127`	`this._oLocaleData = new LocaleData(this._oLocale);`
`128`	`128`