Specify classes or id's for elements about ticketit

[thekordy]

You are very right 👍

[fishmad]

Hardcoded CSS is building fast we need to agree on some naming conventions for TicketIT id and class tags and follow the pattern throughout, also need a custom stylesheet implemented.

[fusiondesign]

I don't know if most email clients support stylesheets. Every Open Source template I've ever seen has inline styles with comments like "" and stuff.

This was an open source template as well that I converted into a theme.

[fusiondesign]

Which is why database setting variable in a ticketit_email_templates database may be wise

[fusiondesign]

Then we can have multiple "themes" that use the same template

[fusiondesign]

That said, I totally agree on id's and classes throughout the normal app.

[fishmad]

Agree, emails should have embedded styles - linked stylesheets in emails break to often and probably end up in clients spam folders.

Are we going to just wing-it with class names or follow the existing TitcketIT folder structure?

should we wrap each View file within its own custom div and ID or do integrate into existing the bootstrap something like below:

vendor\kordy\ticketit\src\Views\tickets\index.blade.php

Before: <div class="panel panel-default"> // Line 1 in view file
After: <div id="ticketit" class="panel panel-default ticketit-index" >

vendor\kordy\ticketit\src\Views\tickets\create.blade.php

Before: <div class="well bs-component"> // Line 1 in view file
After: <div id="ticketit" class="well bs-component ticketit-create">

vendor\kordy\ticketit\src\Views\tickets\show.blade.php (\partials\ticketit_body.blade.php)

Before: <div class="panel panel-default"> // Line 1 in view file
After: <div id="ticketit" class="panel panel-default ticketit-show ticket-{{ $ticket->id }}" >

OR assign ID once in Master, then use classes only in views

vendor\kordy\ticketit\src\Views\index.blade.php

@section('content')
Before: <div class="row">
After: <div class="row" id="ticketit">

vendor\kordy\ticketit\src\Views\tickets\index.blade.php

Before: <div class="panel panel-default"> // Line 1 in view file
After: <div class="panel panel-default ticketit-index" >

vendor\kordy\ticketit\src\Views\tickets\create.blade.php

Before: <div class="well bs-component"> // Line 1 in view file
After: <div class="well bs-component ticketit-create">

vendor\kordy\ticketit\src\Views\tickets\show.blade.php (\partials\ticketit_body.blade.php)

Before: <div class="panel panel-default"> // Line 1 in view file
After: <div class="panel panel-default ticketit-show ticket-{{ $ticket->id }}" >

[thekordy]

assign ID once in Master, then use classes only in views

I am with this option 👍