yii2-debug
yii2-debug copied to clipboard
Allow to configure toolbar position
Q | A |
---|---|
Is bugfix? | ❌ |
New feature? | ✔️ |
Breaks BC? | ❌ |
Tests pass? | ✔️ |
Add Module::$toolbarPosition option. It allows to configure toolbar position css class except of hardcode it to 'bottom' like
$config['modules']['debug'] = [
'class' => 'yii\debug\Module',
'toolbarPosition' => 'top',
];
Done.
I also noticed .yii-debug-toolbar_position_top
css name seems not perfect as it place toolbar below page footer.
May be rename somehow (.yii-debug-toolbar_position_footer
)?
Whatever works.
⚠️ Note to myself: actually check it in the browser before merging.
Checked it, needs extra work:
- There's no ready to use position values you can configure.
top
doesn't make sense now. - Collapsing toolbar doesn't work well with non-default value.
1. There's no ready to use position values you can configure. `top` doesn't make sense now.
What's wrong with top
value?
2. Collapsing toolbar doesn't work well with non-default value.
How can I check it?
What's wrong with top value?
It results in toolbar being rendered at the bottom under content and not fixed.
How can I check it?
Set value to top
. Try collapsing the toolbar.
@sasha-x do you want to finish it?
Yes, I need 2-3 days, please.
Hello, sorry for delay.
What's wrong with top value? It results in toolbar being rendered at the bottom under content and not fixed.
It was exactly what I need in my project (to free right-bottom fixed position of viewport for other ui elements).
Current name of that slyle (top
) is wrong. I suggest to rename it. For example name it as footer
.
Is that ok?
Collapsing toolbar doesn't work well with non-default value.
Yes, I see. I can try to fix that.
It was exactly what I need in my project (to free right-bottom fixed position of viewport for other ui elements).
I wonder if that's a common need. Not sure it will be need for anyone else.
This style was made sometime and currently unused anywhere.
If no, what additional style will be useful? May be left-bottom-fixed position?
Top-fixed position should be useful.
Do you mean overlapping of navbar line?
Fixed one. Yes.
Css for top-right-fixed toolbar position added.
I name it with upper
postfix (top
is already in use).
Review it, pls.
Thank you!