joplin icon indicating copy to clipboard operation
joplin copied to clipboard

Notebook icon does not show in 'Move to Notebook' dropdown menu

Open mak2002 opened this issue 2 years ago • 16 comments

Notebook icons are not appearing in 'Move to Notebook' dropdown menu(below image). bug-pic

Environment

Joplin version: 2.7.15 (prod) Platform: Linux OS specifics: Ubuntu 20.04 LTS

Steps to reproduce

  1. Make a notebook with some icon
  2. Make a note/todo and right click on it.
  3. On clicking dropdown menu it shows list of notebook without icons.

Describe what you expected to happen

Notebook icons should be displayed in the drop down list.

mak2002 avatar Mar 18 '22 03:03 mak2002

I think adding "desktop" as a label would be helpful as when I tested this on android tablet it is showing the icons. It would be great if anybody can confirm this.

mak2002 avatar Mar 18 '22 12:03 mak2002

I am trying to work on this issue, but I am facing a problem. When the emoji is a user uploaded image, I am getting a base64 string instead of the image, obviously. Currently, I can not think of a way to render an image as an emoji (without the need for an image tag or a react node) One workaround is to use something like 🖼️ for custom images. As for android, currently, nothing is shown for custom images. Kindly push me in the right direction. image image image

husain3012 avatar Mar 19 '22 16:03 husain3012

Is this bug for mobile or desktop?

entrymaster avatar Mar 23 '22 07:03 entrymaster

Is this bug for mobile or desktop?

desktop, but for some reason it's labelled as mobile

husain3012 avatar Mar 23 '22 09:03 husain3012

I have observed that on Mobile App there isn't an option for adding a icon to notebook while creating a notebook. Also no icon appears in "Move Notebook" modal. So do this issue aims to add icon feature in mobile too or seperate issue can be created for that ?

entrymaster avatar Mar 25 '22 15:03 entrymaster

I had initially created this issue for icon not showing on desktop, but didn't know that we can't add icons for notebook in mobile. Thanks for letting us know.

As far as creating a new issue goes, let's see what @laurent22 has to say.

mak2002 avatar Mar 25 '22 16:03 mak2002

Can i work on this issue?

yes

roman-r-m avatar Mar 29 '22 20:03 roman-r-m

I have observed that on Mobile App there isn't an option for adding a icon to notebook while creating a notebook. Also no icon appears in "Move Notebook" modal. So do this issue aims to add icon feature in mobile too or seperate issue can be created for that ?

@roman-r-m Can u please confirm this ?

entrymaster avatar Apr 03 '22 18:04 entrymaster

I had initially created this issue for icon not showing on desktop, but didn't know that we can't add icons for notebook in mobile. Thanks for letting us know.

As far as creating a new issue goes, let's see what @laurent22 has to say.

Is the issue in desktop or mobile currently ? @laurent22 @roman-r-m

BalanaguYashwanth avatar Sep 10 '22 11:09 BalanaguYashwanth

Is the issue in desktop or mobile currently ? @laurent22 @roman-r-m

I don't know but it's very easy to check isn't it?

roman-r-m avatar Sep 10 '22 11:09 roman-r-m

I don't know but it's very easy to check isn't it? @roman-r-m

To check in mobile version, i can download app from google playstore and check directly ?

BalanaguYashwanth avatar Sep 10 '22 12:09 BalanaguYashwanth

Yes. Or you can also build from sources, whatever you prefer

roman-r-m avatar Sep 10 '22 20:09 roman-r-m

Yes. Or you can also build from sources, whatever you prefer

Thanks @roman-r-m

BalanaguYashwanth avatar Sep 11 '22 05:09 BalanaguYashwanth

Finally i added this feature for desktop, so can i raise PR ?

Kindly express your thoughts.

Screenshot 2022-09-11 at 3 15 49 PM

BalanaguYashwanth avatar Sep 11 '22 09:09 BalanaguYashwanth

Finally i added this feature for desktop, so can i raise PR ?

Of course. Make sure to check the PR guidelines first.

And please stop @-ing people, this does not help at all

roman-r-m avatar Sep 11 '22 09:09 roman-r-m

And please stop @-ing people, this does not help at all

Ok

BalanaguYashwanth avatar Sep 11 '22 10:09 BalanaguYashwanth