assign a symbol to a module

ianperrin

Hi @Plati

The solution depends on how you want to display the table.

Do you want

3 columns in the following order: | Icon | Message | Time |
Or 2 columns laid out as follows: | Message Icon | Time |

Plati

Hi @ianperrin

| Icon | Message | Time | should be ok

ianperrin

@Plati said in assign a symbol to a module:

Hi @ianperrin

| Icon | Message | Time | should be ok

In that case, try this


		var iconCell = document.createElement("td");

		var icon =  document.createElement("i");
		if (this.messages[i].type === "INFO"){
			icon.className = "fa fa-info";
		}
		else if (this.messages[i].type === "WARNING"){
			icon.className = "fa fa-exclamation";
		}
		else if (this.messages[i].type === "ERROR"){
			icon.className = "fa fa-exclamation-triangle";
		}
		else {
			icon.className = "fa fa-question";
		}

		iconCell.appendChild(icon);
		callWrapper.appendChild(iconCell);
		
		//Set caller of row
		var caller =  document.createElement("td");
		caller.innerHTML = "[" + this.messages[i].type + "] " + this.messages[i].message;

ianperrin

You could also consider making the icon configurable via the options,

First add the default values for the options

	defaults: {
		max: 5,
		format: false,
		types: {
			INFO: "dimmed",
			WARNING: "normal",
			ERROR: "bright"
		},
		icons: {
			INFO: "info",
			WARNING: "exclamation",
			ERROR: "exclamation-triangle"
		}
	},

Then change the code to create the icon (and for bonus points, set class for the icon to the same as the message.

		var iconCell = document.createElement("td");
		var icon =  document.createElement("i");
		if(this.config.icons.hasOwnProperty(this.messages[i].type)){
			icon.classList.add("fa", "fa-" + this.config.icons[this.messages[i].type]);
		} else {
			icon.classList.add("fa", "fa-question");
		}
		if(this.config.types.hasOwnProperty(this.messages[i].type)){
			icon.classList.add(this.config.types[this.messages[i].type]);
		}
		iconCell.appendChild(icon);
		callWrapper.appendChild(iconCell);

Warning - this code was entered directly here in the comment so may contain TyoPs ;)

Plati

@ianperrin nice! I still have a lot to learn. I’m just beginning to learn JS, etc.

Now is almost perfect:

Now only center the icons and reduce the spacing between the lines.

ianperrin

@Plati glad to help - and we’re all learning here so don’t worry!

Fontawesome provides lots of support for managing the display of icons, so if the icons are throwing the alignment in the table out try making the icons Fixed Width by adding “fa-fw” to the icon e.g. icon.classList.add("fa", "fa-fw", "fa-" …

Alternatively, set the icon font-size to the same as the message, by using the inbuilt MM styles, e.g. iconCell.classList.add("small");

or a combination of both :)

dominic

@yawns Hi, Thanks it works.
I have an other question.

how can I get the values , that they will displayed in two lines?

ianperrin

Hi @dominic

The simplest approach would be to add a “br” element between the temperature and humidity.

However, I would consider putting them in an unordered list and take advantage of the inbuilt layout controls within FontAwesome. E.g.

getDom: function() {
	var wrapper = document.createElement("div");
	if(this.dataFile){

		var humidityRegExp = /Humidity = (.*?) %/ig;
		var humidity = humidityRegExp.exec(this.dataFile)[1];

		var temperatureRegExp = /Temperature = (.*?) *C/ig;
		var temperature = temperatureRegExp.exec(this.dataFile)[1];

		var list = document.createElement("ul");
		list.classList.add("fa-ul");

		// add temperature
		var temperature_item = document.createElement("li");
		var temperature_symbol =  document.createElement("i");
		temperature_symbol.classList.add("fa", "fa-li", "fa-home");
		temperature_item.appendChild(temperature_symbol);
		temperature_item.appendChild(document.createTextNode(" " + temperature + "&deg;C"));
		list.appendChild(temperature_item);

		// add humidity 
		var humidity_item = document.createElement("li");
		var humidity_symbol =  document.createElement("i");
		humidity_symbol.classList.add("fa", "fa-li", "fa-tint");
		humidity_item.appendChild(humidity_symbol);
		humidity_item.appendChild(document.createTextNode(" " + humidity + "%"));
		list.appendChild(humidity_item);

		wrapper.appendChild(list);

	} else {
		wrapper.innerHTML = "No data";
	}
	return wrapper;
},

dominic

Hi ianperrin, thank you :)

Do you knwo, how can i erase the “*” befor the “°C” ?

Plati

@ianperrin you are wizard :) big thank you, now it works excellent

@dominic try delete " * " from var temperatureRegExp = /Temperature = (.*?) *C/ig;
try it:

var temperatureRegExp = /Temperature = (.*?) C/ig;