B00merang-Project / macOS

macOS theme for Gnome and GTK-based desktops
http://b00merang.weebly.com/macos.html
GNU General Public License v3.0
744 stars 118 forks source link

fix cinnamon menu margin (so search box does not abut right edge) #90

Closed VK2BEA closed 4 years ago

VK2BEA commented 4 years ago

Yes, minor/trivial but the search box against the right edge is visually annoying 8-(

Elbullazul commented 4 years ago

can you apply it specifically to the menu StEntry?

VK2BEA commented 4 years ago

Well it doesn't work in StEntry (this already has a padding entry).I can change the entry in ... .popup-menu,.menu { but I'm not sure if this has some other side-effect ( ...  I don't know what .popup-menu is used for) (Let me know if you want me to just adjust the padding-right here.Michael. 

On Monday, March 2, 2020, 01:04:25 PM EST, Christian Medel <notifications@github.com> wrote:  

can you apply it specifically to the menu StEntry?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

Elbullazul commented 4 years ago

could you try and submit

.menu StEntry {
    margin-right: 10px;
}
VK2BEA commented 4 years ago

Yes, although 5px looks better IMHO (with the right of the entry box lining up with the right edge of the slider) On Monday, March 2, 2020, 07:36:48 PM EST, Christian Medel notifications@github.com wrote:

could you try and submit .menu StEntry { margin-right: 10px; } — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

VK2BEA commented 4 years ago

Actually this is not as good as adjusting the menu padding when there is no slider...   see belowThe first is margin-right in the StEntry .menu, the second using the padding-right adjustment in the menu,  Michael On Monday, March 2, 2020, 07:41:58 PM EST, Michael K vk2bea@yahoo.com wrote:

Yes, although 5px looks better IMHO (with the right of the entry box lining up with the right edge of the slider) On Monday, March 2, 2020, 07:36:48 PM EST, Christian Medel notifications@github.com wrote:

could you try and submit .menu StEntry { margin-right: 10px; } — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

Elbullazul commented 4 years ago

I think the screenshots didn't upload dorrectly. Would you mind uploading them from github directly?

Elbullazul commented 4 years ago

I'll merge this today, but only for the main menu (suggested fix applies to context menus as well)