1091

I am writing an Angular application and I have an HTML response I want to display.

How do I do that? If I simply use the binding syntax {{myVal}} it encodes all HTML characters (of course).

I need somehow to bind the innerHTML of a div to the variable value.

1

25 Answers 25

1646

The correct syntax is the following:

<div [innerHTML]="theHtmlString"></div>

Documentation Reference

16
  • 24
    Is there any way I can force angular to run its binding on the elements of that innerHTML? I need to use an <a [router-link]="..."></a>, and want to provide that from external html.
    – dessalines
    Commented Apr 29, 2016 at 22:04
  • 6
    It renders the string in my case, but does something to the markup. Seems to have stripped out attributes on markup . I'm on 2.4.6
    – crthompson
    Commented Mar 20, 2017 at 23:10
  • 2
    @thouliha did you ever find an answer? is there any way to force angular to reevaluate the bindings after the injection? all my bindings and templates are broken within the innerHtml portion
    – jgritten
    Commented Jan 28, 2019 at 18:46
  • 2
    The other answers stackoverflow.com/a/41089093/1225421 and stackoverflow.com/a/41121006/1225421 solve the security HTML sanitzer problem.
    – Alex P.
    Commented Apr 8, 2019 at 13:35
  • 2
    @jgritten Have a look at npmjs.com/package/ngx-dynamic-hooks Commented Aug 31, 2020 at 12:30
398

Angular 2.0.0 and Angular 4.0.0 final

For safe content just

<div [innerHTML]="myVal"></div>

DOMSanitizer

Potential unsafe HTML needs to be explicitly marked as trusted using Angulars DOM sanitizer so doesn't strip potentially unsafe parts of the content

<div [innerHTML]="myVal | safeHtml"></div>

with a pipe like

@Pipe({name: 'safeHtml'})
export class Safe {
  constructor(private sanitizer:DomSanitizer){}

  transform(style) {
    return this.sanitizer.bypassSecurityTrustHtml(style);
    //return this.sanitizer.bypassSecurityTrustStyle(style);
    // return this.sanitizer.bypassSecurityTrustXxx(style); - see docs
  }
}

See also In RC.1 some styles can't be added using binding syntax

And docs: https://angular.io/api/platform-browser/DomSanitizer

Security warning

Trusting user added HTML may pose a security risk. The before mentioned docs state:

Calling any of the bypassSecurityTrust... APIs disables Angular's built-in sanitization for the value passed in. Carefully check and audit all values and code paths going into this call. Make sure any user data is appropriately escaped for this security context. For more detail, see the Security Guide.

Angular markup

Something like

class FooComponent {
  bar = 'bar';
  foo = `<div>{{bar}}</div>
    <my-comp></my-comp>
    <input [(ngModel)]="bar">`;

with

<div [innerHTML]="foo"></div>

won't cause Angular to process anything Angular-specific in foo. Angular replaces Angular specific markup at build time with generated code. Markup added at runtime won't be processed by Angular.

To add HTML that contains Angular-specific markup (property or value binding, components, directives, pipes, ...) it is required to add the dynamic module and compile components at runtime. This answer provides more details How can I use/create dynamic template to compile dynamic Component with Angular 2.0?

14
  • 21
    This should be the answer. Pay attention to the two lines that are commented out. It is actually the second one that handles HTML.
    – crthompson
    Commented Mar 20, 2017 at 23:28
  • 10
    be sure to import { BrowserModule, DomSanitizer } from '@angular/platform-browser'
    – crthompson
    Commented Mar 20, 2017 at 23:29
  • 6
    Also import { Pipe } from '@angular/core'
    – Appulus
    Commented Mar 30, 2017 at 9:37
  • 2
    Use DomSanitizer instead Commented Jul 16, 2017 at 9:31
  • 2
    This answer solves the problems generated by HTML consider unsafe. I managed to define text color and add a youtube iframe with it. You cannot achieve this by simply setting the innerHTML like it is descibed in the other answers.
    – Alex P.
    Commented Apr 8, 2019 at 13:31
196

[innerHtml] is great option in most cases, but it fails with really large strings or when you need hard-coded styling in html.

I would like to share other approach:

All you need to do, is to create a div in your html file and give it some id:

<div #dataContainer></div>

Then, in your Angular 2 component, create reference to this object (TypeScript here):

import { Component, ViewChild, ElementRef } from '@angular/core';

@Component({
    templateUrl: "some html file"
})
export class MainPageComponent {

    @ViewChild('dataContainer') dataContainer: ElementRef;

    loadData(data) {
        this.dataContainer.nativeElement.innerHTML = data;
    }
}

Then simply use loadData function to append some text to html element.

It's just a way that you would do it using native javascript, but in Angular environment. I don't recommend it, because makes code more messy, but sometimes there is no other option.

See also Angular 2 - innerHTML styling

17
  • 2
    Other solutions save string to html attribute first, than loads html. Saving large string to attribute causes browser freezes or even crashes. My solution omits this "attribute part"
    – Piotrek
    Commented Aug 8, 2016 at 10:36
  • 1
    Yep, otherwise I wouldn't add my answer here
    – Piotrek
    Commented Aug 8, 2016 at 10:47
  • 29
    [innerHtml] removes styling hard-coded in the Html. In order to integrate a wysiwyg editor, I had to use the approach listed here. Commented Aug 14, 2016 at 6:11
  • 1
    This is useful for generating content that will go into an HTML email where inline styling is unfortunately still necessary. Other methods using interpolation removed the inline styles.
    – Frank
    Commented Aug 31, 2016 at 17:35
  • 1
    Can this @ViewChild work for multiple div s? and if so, how?
    – Guram
    Commented Feb 22, 2017 at 12:07
69

On [email protected]:

Html-Binding will not work when using an {{interpolation}}, use an "Expression" instead:

invalid

<p [innerHTML]="{{item.anleser}}"></p>

-> throws an error (Interpolation instead of expected Expression)

correct

<p [innerHTML]="item.anleser"></p>

-> this is the correct way.

you may add additional elements to the expression, like:

<p [innerHTML]="'<b>'+item.anleser+'</b>'"></p>

hint

HTML added using [innerHTML] (or added dynamically by other means like element.appenChild() or similar) won't be processed by Angular in any way except sanitization for security purposed.
Such things work only when the HTML is added statically to a components template. If you need this, you can create a component at runtime like explained in How can I use/create dynamic template to compile dynamic Component with Angular 2.0?

1
  • 3
    The third example not working. The expression is not evaluate. The output is simply string... Any other way to combile trustedHTML with another tags elements ? Commented Dec 6, 2016 at 8:41
32

Using [innerHTML] directly without using Angular's DOM sanitizer is not an option if it contains user-created content. The safeHtml pipe suggested by @GünterZöchbauer in his answer is one way of sanitizing the content. The following directive is another one:

import { Directive, ElementRef, Input, OnChanges, Sanitizer, SecurityContext,
  SimpleChanges } from '@angular/core';

// Sets the element's innerHTML to a sanitized version of [safeHtml]
@Directive({ selector: '[safeHtml]' })
export class HtmlDirective implements OnChanges {
  @Input() safeHtml: string;

  constructor(private elementRef: ElementRef, private sanitizer: Sanitizer) {}

  ngOnChanges(changes: SimpleChanges): any {
    if ('safeHtml' in changes) {
      this.elementRef.nativeElement.innerHTML =
        this.sanitizer.sanitize(SecurityContext.HTML, this.safeHtml);
    }
  }
}

To be used

<div [safeHtml]="myVal"></div>
2
  • 1
    @ObasiObenyOj you can still do that without the using of a separate directive if is a limited case, constructor( private sanitizer: Sanitizer) {} and bind the result into whatever you need, also the usage of ElementRef is strongly unsuggested.
    – Valex
    Commented Feb 3, 2017 at 11:58
  • Does it work with input and text-area elements? Commented Feb 15, 2021 at 16:16
29

Please refer to other answers that are more up-to-date.

This works for me: <div innerHTML = "{{ myVal }}"></div> (Angular2, Alpha 33)

According to another SO: Inserting HTML from server into DOM with angular2 (general DOM manipulation in Angular2), "inner-html" is equivalent to "ng-bind-html" in Angular 1.X

2
  • 4
    Use the [property] binding syntax instead of the {{interpolation}} Commented Oct 27, 2017 at 9:29
  • also work in my project where dont wont work <div [innerHTML] = "{{ myVal }}"></div>
    – mbielecki1
    Commented Mar 5, 2021 at 8:17
27

Just to make for a complete answer, if your HTML content is in a component variable, you could also use:

<div [innerHTML]=componentVariableThatHasTheHtml></div>
16

Short answer was provided here already: use <div [innerHTML]="yourHtml"> binding.

However the rest of the advices mentioned here might be misleading. Angular has a built-in sanitizing mechanism when you bind to properties like that. Since Angular is not a dedicated sanitizing library, it is overzealous towards suspicious content to not take any risks. For example, it sanitizes all SVG content into empty string.

You might hear advices to "sanitize" your content by using DomSanitizer to mark content as safe with bypassSecurityTrustXXX methods. There are also suggestions to use pipe to do that and that pipe is often called safeHtml.

All of this is misleading because it actually bypasses sanitizing, not sanitizing your content. This could be a security concern because if you ever do this on user provided content or on anything that you are not sure about — you open yourself up for a malicious code attacks.

If Angular removes something that you need by its built-in sanitization — what you can do instead of disabling it is delegate actual sanitization to a dedicated library that is good at that task. For example — DOMPurify.

I've made a wrapper library for it so it could be easily used with Angular: https://github.com/TinkoffCreditSystems/ng-dompurify

EDIT: Moved here: https://github.com/taiga-family/ng-dompurify

It also has a pipe to declaratively sanitize HTML:

<div [innerHtml]="value | dompurify"></div>

The difference to pipes suggested here is that it actually does do the sanitization through DOMPurify and therefore work for SVG.

EDIT: Angular no longer sanitizes CSS as of Ivy renderer so below info, kept for history sake, is irrelevant:

One thing to keep in mind is DOMPurify is great for sanitizing HTML/SVG, but not CSS. So you can provider Angular's CSS sanitizer to handle CSS:

import {NgModule, ɵ_sanitizeStyle} from '@angular/core';
import {SANITIZE_STYLE} from '@tinkoff/ng-dompurify';

@NgModule({
    // ...
    providers: [
        {
            provide: SANITIZE_STYLE,
            useValue: ɵ_sanitizeStyle,
        },
    ],
    // ...
})
export class AppModule {}

It's internal — hence ɵ prefix, but this is how Angular team use it across their own packages as well anyway. That library also works for Angular Universal and server side rendering environment.

12

I apologize if I am missing the point here, but I would like to recommend a different approach:

I think it's better to return raw data from your server side application and bind it to a template on the client side. This makes for more nimble requests since you're only returning json from your server.

To me it doesn't seem like it makes sense to use Angular if all you're doing is fetching html from the server and injecting it "as is" into the DOM.

I know Angular 1.x has an html binding, but I have not seen a counterpart in Angular 2.0 yet. They might add it later though. Anyway, I would still consider a data api for your Angular 2.0 app.

I have a few samples here with some simple data binding if you are interested: http://www.syntaxsuccess.com/viewarticle/angular-2.0-examples

3
  • 30
    There's definitely use cases where you'd want to fetch and display raw html. E.g. fetching a formatted piece of article from remote. Commented Dec 24, 2015 at 2:17
  • 2
    Another often-ignored scenario is protecting the business logic in the template, you sometimes don't want unauthorized users to see the logic you are using to display information, so you would rather prepare the view on server side
    – Ayyash
    Commented Feb 20, 2016 at 11:45
  • 2
    If you are missing the point (which you seem to be by your own admission), then why post a response? Obviously the point of Angular is to use its view engine to bind and render the data. But considering the fact that there are countless applications where an Angular app might be used, it is actually feasible that one or two of them might have the requirement that some of the data that needs to be displayed in your application may already be formatted HTML, and it might just happen to be the case where the developer does not have control over that content. In other words... relevant question.
    – Gregor
    Commented Nov 1, 2017 at 5:50
8

 <div [innerHTML]="HtmlPrint"></div><br>

The innerHtml is a property of HTML-Elements, which allows you to set it’s html-content programatically. There is also a innerText property which defines the content as plain text.

The [attributeName]="value" box bracket , surrounding the attribute defines an Angular input-binding. That means, that the value of the property (in your case innerHtml) is bound to the given expression, when the expression-result changes, the property value changes too.

So basically [innerHtml] allows you to bind and dynamically change the html-conent of the given HTML-Element.

7

Just simply use [innerHTML] attribute in your HTML, something like this below:

<div [innerHTML]="myVal"></div>

Ever had properties in your component that contain some html markup or entities that you need to display in your template? The traditional interpolation won't work, but the innerHTML property binding comes to the rescue.

Using {{myVal}} Does NOT work as expected! This won't pick up the HTML tags like <p>, <strong> etc and pass it only as strings...

Imagine you have this code in your component:

const myVal:string ='<strong>Stackoverflow</strong> is <em>helpful!</em>'

If you use {{myVal}}, you will get this in the view:

<strong>Stackoverflow</strong> is <em>helpful!</em>

but using [innerHTML]="myVal"makes the result as expected like this:

Stackoverflow is helpful!

0
4

You can apply multiple pipe for style, link and HTML as following in .html

<div [innerHTML]="announcementContent | safeUrl| safeHtml">
                    </div>

and in .ts pipe for 'URL' sanitizer

import { Component, Pipe, PipeTransform } from '@angular/core';
import { DomSanitizer } from '@angular/platform-browser';

@Pipe({ name: 'safeUrl' })
export class SafeUrlPipe implements PipeTransform {
    constructor(private sanitizer: DomSanitizer) {}
    transform(url) {
        return this.sanitizer.bypassSecurityTrustResourceUrl(url);
    }
}

pipe for 'HTML' sanitizer

import { Component, Pipe, PipeTransform } from '@angular/core';
import { DomSanitizer } from '@angular/platform-browser';

@Pipe({
    name: 'safeHtml'
})
export class SafeHtmlPipe implements PipeTransform {
    constructor(private sanitized: DomSanitizer) {}
    transform(value) {
        return this.sanitized.bypassSecurityTrustHtml(value);
    }
}

this will apply both without disturbing any style and link click event

2

In Angular 2 you can do 3 types of bindings:

  • [property]="expression" -> Any html property can link to an
    expression. In this case, if expression changes property will update, but this doesn't work the other way.
  • (event)="expression" -> When event activates execute expression.
  • [(ngModel)]="property" -> Binds the property from js (or ts) to html. Any update on this property will be noticeable everywhere.

An expression can be a value, an attribute or a method. For example: '4', 'controller.var', 'getValue()'

Example here

2

We can always pass html content to innerHTML property to render html dynamic content but that dynamic html content can be infected or malicious also. So before passing dynamic content to innerHTML we should always make sure the content is sanitized (using DOMSanitizer) so that we can escaped all malicious content.

Try below pipe:

import { Pipe, PipeTransform } from "@angular/core";
import { DomSanitizer } from "@angular/platform-browser";

@Pipe({name: 'safeHtml'})
export class SafeHtmlPipe implements PipeTransform {
    constructor(private sanitized: DomSanitizer) {
    }
    transform(value: string) {
        return this.sanitized.bypassSecurityTrustHtml(value);
    }
}

Usage:
<div [innerHTML]="content | safeHtml"></div>
0
2

You can also bind the angular component class properties with template using DOM property binding.

Example: <div [innerHTML]="theHtmlString"></div>

Using canonical form like below:

<div bind-innerHTML="theHtmlString"></div>

Angular Documentation: https://angular.io/guide/template-syntax#property-binding-property

See working stackblitz example here

1

Working in Angular v2.1.1

<div [innerHTML]="variable or htmlString">
</div>
1
  • 2
    This produces: <div _ngcontent-luf-0=""></div> for me. The div is empty. Commented Nov 23, 2016 at 15:54
1

I have build below library which will help to rebind html formatted bindings. Please find below steps to use this library. This library basically allows to inject JIT compilter code in AOT

  1. Install library using

    npm i angular-html-recompile
    
  2. Add below code in app.component.html file

    <pk-angular-html-recompile *ngIf="template !== ''" 
                               [stringTemplate]="template" 
                               [data]="dataObject"> 
    </pk-angular-html-recompile>
    
  3. Use below code in app.component.ts file

    import { Component, OnInit, ViewChild } from '@angular/core';
    import { AngularHtmlRecompileComponent, AngularHtmlRecompileService } from 'angular-html-recompile';
    
    @Component({
        selector: 'app-root',
        templateUrl: './app.component.html',
        styleUrls: ['./app.component.scss']
    })
    export class AppComponent implements OnInit {
    
        @ViewChild(AngularHtmlRecompileComponent, { static: true }) comp !: AngularHtmlRecompileComponent;
    
        constructor(
            private angularHtmlRecompileService: AngularHtmlRecompileService) {
        }
    
        public dataObject: any;
    
        public template = `<div class="login-wrapper" fxLayout="row" fxLayoutAlign="center center">
        <mat-card class="box">
            <mat-card-header>
            <mat-card-title>Register</mat-card-title>
            </mat-card-header>
            <form class="example-form">
            <mat-card-content>
                <mat-form-field class="example-full-width">
                <input matInput placeholder="Username" [value]="Username" (keydown)="onControlEvent($event,'Username')">
                </mat-form-field>
                <mat-form-field class="example-full-width">
                <input matInput placeholder="Email" [value]="Email" (keydown)="onControlEvent($event,'Email')">
                </mat-form-field>
                <mat-form-field *ngIf="isShow" class="example-full-width">
                <input matInput placeholder="Password" [value]="Password" (keydown)="onControlEvent($event,'Password')">
                </mat-form-field>
                <mat-form-field class="example-full-width">
                <mat-label>Choose a role...</mat-label>
                <mat-select (selectionChange)="onControlEvent($event, 'selectedValue')">
                    <mat-option [value]="roles" *ngFor="let roles of Roles">{{roles}}
                    </mat-option>
                </mat-select>
                </mat-form-field>
            </mat-card-content>
            <button mat-stroked-button color="accent" class="btn-block" (click)="buttomClickEvent('submit')" >Register</button>
            </form>
        </mat-card>
        </div>`;
    
        ngOnInit(): void {
    
            this.angularHtmlRecompileService.sharedData.subscribe((respose: any) => {
                if (respose) {
                    switch (respose.key) {
                        case `Username`:
                            // Call any method on change of name
                            break;
                        case `Password`:
                            //Update password from main component
                            this.comp[`cmpRef`].instance['Password'] = "Karthik";
                            break;
                        case `submit`:
                            //Get reference of all parameters on submit click
                            //1. respose.data OR
                            //use this.comp[`cmpRef`].instance
                            break;
                        default:
                            break;
                    }
                }
            });
    
            this.prepareData();
    
        }
    
        prepareData() {
    
            //Prepare data in following format only for easy binding
            //Template preparation and data preparation can be done once data received from service
    
            // AngularHtmlRecompileComponent will not be rendered until you pass data 
    
            this.dataObject = [
                { key: 'Username', value: 'Pranay' },
                { key: 'Email', value: '[email protected]', },
                { key: 'Password', value: 'test123', },
                { key: 'Roles', value: ['Admin', 'Author', 'Reader'] },
                { key: 'isShow', value: this.updateView() }
            ];
    
        }
    
        updateView() {
            //Write down logic before rendering to UI to work ngIf directive
            return true;
        }
    }
    
  4. Add module into app.module.ts file

    import { NgModule } from '@angular/core';
    import { BrowserModule } from '@angular/platform-browser';
    
    import { AppComponent } from './app.component';
    import { AngularHtmlRecompileModule } from "angular-html-recompile";
    
    @NgModule({
        declarations: [
            AppComponent
        ],
        imports: [
            BrowserModule,
            AngularHtmlRecompileModule
        ],
        providers: [],
        bootstrap: [AppComponent]
    })
    export class AppModule { }
    
  5. This library supports basic html, Angular material, flex layouts. To use this features install below dependencies

    npm i -s @angular/material @angular/flex-layout
    
0

The way to dynamically add elements to DOM, as explained on Angular 2 doc, is by using ViewContainerRef class from @Angular/core.

What you have to do is to declare a directive that will implement ViewContainerRef and act like a placeholder on your DOM.

Directive

import { Directive, ViewContainerRef } from '@angular/core';

@Directive({
  selector: '[appInject]'
})
export class InjectDirective {

  constructor(public viewContainerRef: ViewContainerRef) { }

}

Then, in the template where you want to inject the component:

HTML

<div class="where_you_want_to_inject">    
  <ng-template appInject></ng-template>
</div>

Then, from the injected component code, you will inject the component containing the HTML you want:

import { Component, OnInit, ViewChild, ComponentFactoryResolver } from '@angular/core';
import { InjectDirective } from '../inject.directive';
import { InjectedComponent } from '../injected/injected.component';

@Component({
  selector: 'app-parent',
  templateUrl: './parent.component.html',
  styleUrls: ['./parent.component.css']
})
export class ParentComponent implements OnInit {

  @ViewChild(InjectDirective) injectComp: InjectDirective;

  constructor(private _componentFactoryResolver: ComponentFactoryResolver) {
  }

  ngOnInit() {
  }

  public addComp() {
    const componentFactory = this._componentFactoryResolver.resolveComponentFactory(InjectedComponent);
    const viewContainerRef = this.injectComp.viewContainerRef;
    const componentRef = viewContainerRef.createComponent(componentFactory);
  }

  public removeComp() {
    const componentFactory = this._componentFactoryResolver.resolveComponentFactory(InjectedComponent);
    const viewContainerRef = this.injectComp.viewContainerRef;
    const componentRef = viewContainerRef.remove();
  }

}

I added a fully working demo app on Angular 2 dynamically add component to DOM demo

0

You can use several approaches to achieve the solution. As already said in the approved answer, you can use:

<div [innerHTML]="myVal"></div>

depending on what you are trying to achieve, you can also try other things like javascript DOM (not recommended, DOM operations are slow):

Presentation

<div id="test"></test>

Component

var p = document.getElementsById("test");
p.outerHTML = myVal;

Property Binding

Javascript DOM Outer HTML

2
  • Regardless of whether DOM operations are slower than angular or not, doing it by using getElementsById or any other selection method is bad because it might capture elements belonging to completely different components if they contain elements with the same id (or other criteria).
    – Aviad P.
    Commented Feb 26, 2019 at 12:57
  • Plus it completely performs outside of any angular-zone so changes will not be picked up. Commented Jun 13, 2019 at 10:05
0

If you want that in Angular 2 or Angular 4 and also want to keep inline CSS then you can use

<div [innerHTML]="theHtmlString | keepHtml"></div>
3
  • 1
    This gave me an error ` Uncaught (in promise): Error: Template parse errors: The pipe 'keepHtml' could not be found`
    – Praveen
    Commented Aug 28, 2019 at 9:15
  • import {Pipe,PipeTransform} from "@angular/core";
    – Jay Momaya
    Commented Sep 19, 2019 at 12:18
  • @JayMomaya you may missing the directive keepHtml
    – Anh Hoang
    Commented Jun 29, 2022 at 3:36
0

Just to post a little addition to all the great answers so far: If you are using [innerHTML] to render Angular components and are bummed about it not working like me, have a look at the ngx-dynamic-hooks library that I wrote to address this very issue.

With it, you can load components from dynamic strings/html without compromising security. It actually uses Angular's DOMSanitizer just like [innerHTML] does as well, but retains the ability to load components (in a safe manner).

See it in action in this Stackblitz.

0

You can use the Following two ways.

<div [innerHTML]="myVal"></div>

or

<div innerHTML="{{myVal}}"></div>
0

Angular 2+ supports an [innerHTML] property binding that will render HTML. If you were to otherwise use interpolation, it would be treated as a string.

Into .html file

<div [innerHTML]="theHtmlString"></div>

Into .ts file

theHtmlString:String = "enter your html codes here";
0

First create a Pipe.

import { Pipe, PipeTransform } from '@angular/core';
import { DomSanitizer, SafeHtml } from '@angular/platform-browser';

@Pipe({
name: 'sanitizer',
})

export class SanitizerUrlPipe implements PipeTransform {
constructor(private sanitizer: DomSanitizer) {}

transform(value: string, args: any = 'SafeURL'): SafeUrl | SafeHtml {
    if (args == 'SafeHtml') {
        return this.sanitizer.bypassSecurityTrustHtml(value);
    } 
  }
}

Then, use the pipe in your HTML template like this.

<div [innerHtml]="data | sanitizer : 'SafeHtml'"></div>
-2

If you have templates in your angular (or whatever framework) application, and you return HTML templates from your backend through a HTTP request/response, you are mixing up templates between the frontend and the backend.

Why not just leave the templating stuff either in the frontend (i would suggest that), or in the backend (pretty intransparent imo)?

And if you keep templates in the frontend, why not just respond with JSON for requests to the backend. You do not even have to implement a RESTful structure, but keeping templates on one side makes your code more transparent.

This will pay back when someone else has to cope with your code (or even you yourself are re-entering your own code after a while)!

If you do it right, you will have small components with small templates, and best of all, if your code is imba, someone who doesn't know coding languages will be able to understand your templates and your logic! So additionally, keep your functions/methods as small you can. You will eventually find out that maintaining, refactoring, reviewing, and adding features will be much easier compared to large functions/methods/classes and mixing up templating and logic between the frontend and the backend - and keep as much of the logic in the backend if your frontend needs to be more flexible (e.g. writing an android frontend or switching to a different frontend framework).

Philosophy, man :)

p.s.: you do not have to implement 100% clean code, because it is very expensive - especially if you have to motivate team members ;) but: you should find a good balance between an approach to cleaner code and what you have (maybe it is already pretty clean)

check the book if you can and let it enter your soul: https://de.wikipedia.org/wiki/Clean_Code

1
  • -2 votes. Hah. Ok I will vote for this answer, just because it is obvious that other answers are almost the same. Although this answer doen't provide clean solution it is answer, that puts you in state of thinking, and not copy pasting someones solution.____ In my case I couldn't use this solution, because I had library on the backend that returns code that I have to display as inner html.If I have to move everything from that library, probably would cost me week or two. Sometimes is not possible. But may be I will move only the styles on client side, which may or may not solve my problem.
    – makkasi
    Commented Dec 1, 2020 at 18:44

Not the answer you're looking for? Browse other questions tagged or ask your own question.