Skip to main content
Topic: Index.css - Minor issue with curly bracket spacing (Read 2103 times) previous topic - next topic
0 Members and 1 Guest are viewing this topic.

Index.css - Minor issue with curly bracket spacing

Starts about line 1511 in index.css. Issue is the spaceing between .letterspacing and the '{'.  


Currently is...
Code: [Select]
/* Basic cells. */
.table_grid td, .letterspacing{
    padding: 5px 8px;
    border-bottom: 1px solid #ccc;
}

Should be...
Code: [Select]
/* Basic cells. */
.table_grid td, .letterspacing {
    padding: 5px 8px;
    border-bottom: 1px solid #ccc;
}


Re: Index.css - Minor issue with curly bracket spacing

Reply #1

Should have no effect, so not really a bug.
I've seen many people use one line and no spacing.

Code: [Select]
.table_grid td, .letterspacing{padding:5px 8px;border-bottom:1px solid #ccc;}

Re: Index.css - Minor issue with curly bracket spacing

Reply #2

True dat.  :)  Simply an issue and mostly cosmetic, but I could not determine a good place to put the post. Maybe someone can move this to the Theme section or delete the post if they want?
Last Edit: February 13, 2016, 10:58:47 am by Lou

Re: Index.css - Minor issue with curly bracket spacing

Reply #3

I guess bug reports is fine. ;)
Not something I'd fix in a micro, but something to have a look in 1.1. :)
Bugs creator.
Features destroyer.
Template killer.

Re: Index.css - Minor issue with curly bracket spacing

Reply #4

@Lou, submit a pull-request :D. We need more people committing.

 

Re: Index.css - Minor issue with curly bracket spacing

Reply #5

Well, I have just been traumatized on a Sunday by GitHub by creating an account and forking Elkarte over to my account.

I am fairly certain God is punishing me at this point.  :)

Re: Index.css - Minor issue with curly bracket spacing

Reply #6

Lucky me.  :)   It looks like the errant bracket spacing has already been fixed in the repository. I guess that change was made after I had made the zip download that I installed Elkarte with.  I suppose I could submit the changes for the Member Options and see if they are acceptable. 

Re: Index.css - Minor issue with curly bracket spacing

Reply #7

LOL ... Another happy GIT user :P

That bracket could have been fixed in the development branch by me, I know I did some CSS reformatting / layout stuff for 1.1