Skip to content

Commit

Permalink
More compact rendering of navigation tree
Browse files Browse the repository at this point in the history 
The spacing is not really needed here.

Fixes #12166

Signed-off-by: Michal Čihař <michal@cihar.com>
  • Loading branch information
@nijel
nijel committed Apr 6, 2016
1 parent 9f6a146 commit ed63c26
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 2 deletions.
1 change: 1 addition & 0 deletions ChangeLog
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ phpMyAdmin - ChangeLog
+ issue #11962 Remove embedded PHP libraries, use composer to install them
+ issue #12017 Cannot easily select multiple tables when exporting
+ issue #12047 Add javascript filtering for databases
- issue #12166 More compact rendering of navigation tree

4.6.1 (not yet released)
- issue #12120 PMA_Util not found in insert_edit.lib.php
Expand Down
1 change: 0 additions & 1 deletion themes/original/css/navigation.css.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,6 @@
}
#pma_navigation_tree li {
white-space: nowrap;
padding-bottom: 4px;
clear: both;
min-height: 16px;
}
Expand Down
1 change: 0 additions & 1 deletion themes/pmahomme/css/navigation.css.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,6 @@
#pma_navigation_tree li,
#pma_navigation_tree li.fast_filter {
white-space: nowrap;
padding-bottom: 4px;
clear: both;
min-height: 16px;
}
Expand Down

3 comments on commit ed63c26

@ibennetch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm probably just not used to it yet, but this feels a bit too compact. I'm going to give if a few days and see if I get more used to it.

@nijel
Copy link
Contributor Author

@nijel nijel commented on ed63c26 Apr 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to revert if needed, I prefer that more things fit into screen :-).

@madhuracj
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me as well. Looks more like the original theme. I'd also give it a few days to see whether I get used to it.

Please sign in to comment.