Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notifications: Add support for a bottom position option #7111

Merged
merged 1 commit into from
Feb 18, 2018
Merged

Notifications: Add support for a bottom position option #7111

merged 1 commit into from
Feb 18, 2018

Conversation

jaszhix
Copy link
Contributor

@jaszhix jaszhix commented Dec 1, 2017

@jaszhix jaszhix changed the title Notifications: Add support for a bottom position option [3.8] Notifications: Add support for a bottom position option Dec 1, 2017
@Marcosdw
Copy link

Marcosdw commented Dec 2, 2017

Hello,

i agree totaly


Sorry, i use a oneline translator French/English

@MegaV0lt
Copy link

When willit be released?

@jaszhix
Copy link
Contributor Author

jaszhix commented Dec 14, 2017

Its for CInnamon 3.8, so probably sometime next spring/summer.

@Marcosdw
Copy link

Is there an alternative before cinnamon 3.8?

@jaszhix
Copy link
Contributor Author

jaszhix commented Dec 14, 2017

Not that I know of. If you are feeling confident in editing code and are okay with possibly having to re-install Cinnamon, you can try this:

Look at the PR, in js/ui/messageTray.js, change this line to true and remove this block of code (3 lines). That should get it working on Cinnamon with CJS 3.4+. Copy the file to /usr/share/cinnamon/js/ui/messageTray.js. Restart Cinnamon. If there's a point release and you upgrade, the change will be lost.

@clefebvre clefebvre changed the title [3.8] Notifications: Add support for a bottom position option Notifications: Add support for a bottom position option Dec 19, 2017
@clefebvre clefebvre merged commit bb0afc2 into linuxmint:master Feb 18, 2018
Curly060 pushed a commit to Curly060/Cinnamon that referenced this pull request Jan 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants