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

Problem and features request #3

Closed
Yumega opened this issue Jan 6, 2021 · 12 comments
Closed

Problem and features request #3

Yumega opened this issue Jan 6, 2021 · 12 comments
Labels
enhancement New feature or request

Comments

@Yumega
Copy link

Yumega commented Jan 6, 2021

Firstly,Thanks for your contribution of the codes.

Problem I met:
it works very well if I don't set photos as background,but when I set the background with a photo sliced by the no background js, everything was good on setting and I could see the widget display result on the final setting step,but on the screen it tells Alerts are not supported in a widget.

6CB951B7-FD62-498E-9552-E83C3407C574

Feature request:
The widget looks better if the words color can change into white above the photo background. Like the below ones,it seems that there is a transparent layer between the words and photo, they look nice.

5FF71E9B-FE29-4E83-BDBF-6B1AFFA7206C.

Thanks.

@mvan231
Copy link
Owner

mvan231 commented Jan 6, 2021

For the first bit, the alert is the popup you see during setup. I'm working through some things to make the code more efficient right now actually. But when the widget refreshes that should go away.

As for the font color, what color are you thinking it should be? The images you shared are interesting but I'm not sure what you're meaning to have me change

Also, I assume this is for the calendar widget. Is that correct?

If you have any modifications you've made, you can kindly submit a pull request and I'll review it

@Yumega
Copy link
Author

Yumega commented Jan 7, 2021

I hope the color of all the fonts is white as the fonts on the photo I shared

@mvan231
Copy link
Owner

mvan231 commented Jan 11, 2021

You want all of the font colors to be white?

I have made the ability to use the calendar color as the event name color or not but the font color is still dependent on dark/light mode. Maybe you can show an actual example of the widget and how you'd like to see it. Or you could try modifying it and send me a pull request with the info.

Looking forward to your feedback

@mvan231
Copy link
Owner

mvan231 commented Jan 14, 2021

I think you can check v1.5 that I just released and it might be exactly what you were looking for

@sleepyhellow
Copy link

Hi,
would it be possible to make this as a large widget?

@mvan231
Copy link
Owner

mvan231 commented Jan 17, 2021

Hi,
would it be possible to make this as a large widget?

Are you still talking about the Calendar Indicator widget?

@sleepyhellow
Copy link

sleepyhellow commented Jan 17, 2021

No, I‘m talking about the events you can see. Sorry that was not clear to understand. Would be nice to see the events of the next view weeks (i. e. 14 days).
84B33D3C-97EE-43B2-8E34-CDBBFE4DE2D4

@mvan231
Copy link
Owner

mvan231 commented Jan 18, 2021

No, I‘m talking about the events you can see. Sorry that was not clear to understand. Would be nice to see the events of the next view weeks (i. e. 14 days).
84B33D3C-97EE-43B2-8E34-CDBBFE4DE2D4

The answer to my question then is, yes, not, no.

You're looking to have additional functionality for a large size widget added. That view looks interesting. I'll see what I can do

@mvan231 mvan231 added the enhancement New feature or request label Jan 18, 2021
@sleepyhellow
Copy link

Thanks.

@mvan231
Copy link
Owner

mvan231 commented Jan 20, 2021

I hope the color of all the fonts is white as the fonts on the photo I shared

Did you find the latest version to be what you were looking for?

@mvan231
Copy link
Owner

mvan231 commented Jan 21, 2021

Based on the new improvements that have been added in 1.5 and 1.6 I'm going to close this

@mvan231 mvan231 closed this as completed Jan 21, 2021
@mvan231
Copy link
Owner

mvan231 commented Jan 27, 2021

No, I‘m talking about the events you can see. Sorry that was not clear to understand. Would be nice to see the events of the next view weeks (i. e. 14 days).
84B33D3C-97EE-43B2-8E34-CDBBFE4DE2D4

@sleepyhellow i couldn't find a good way to get in touch with you, but would you want to beta test the new large widget size feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants