> Why is it called `background-face' if it only sets the foreground?
I was following the style of ace-window as I am new to this. I will update that.
> Really small nitpick, but this would read better as a cond. (And that> progn is redundant).
> Is there a reason why this is done like this? Why not just set this> value inside the defvar?
I was adding things to this and defvar only sets it if it is unbound. I figured since this is so easy to read and simple, people might add their own and eval the buffer. This makes it trivial to add more options.
> besides, this package is not a mode.
Totally agree. My thinking drastically changed as I went.
Thanks so much for taking the time to look at my code.
In the elpa wiki it mentioned that if you don't have push rights your code will be added by someone else after two days. I have an account on Savannah but I'm I never saw if this includes push rights to elpa, although I would assume not. Does anyone know what further steps I will need to take?