Skip to content
This repository has been archived by the owner on Oct 24, 2021. It is now read-only.

Add AlwaysOnTop, Hide titlebar and opacity options #31

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eastmane
Copy link

Add settings options for always on top, hide titlebar, opacity to look more 'overlay-ey' :)
Add click to drag and close option on context menu (for when there's no titlebar)

Add click to drag and close option on context menu
@matidfk
Copy link

matidfk commented Jan 2, 2021

This is what I was looking for, if an always on top and hide titlebar option would be implemented that would bring this program from 10/10 to 11.
EDIT: Also some way to perhaps detect when a match ends, that way the program checks still every 30 seconds, but client side, and only once the match has finished sends a request to the server, that way there isnt the abomination of requests there is right now

@HeyM1ke
Copy link
Owner

HeyM1ke commented Jan 3, 2021

This is what I was looking for, if an always on top and hide title bar option would be implemented that would bring this program from 10/10 to 11.
EDIT: Also some way to perhaps detect when a match ends, that way the program checks still every 30 seconds, but client-side, and only once the match has finished sends a request to the server, that way there isn't the abomination of requests there is right now

Sadly if I were to check if a match was over, it would be even more requests, otherwise ill see what I can do with including this into the program. As there will be another massive update soon which can have a lot of conflicts.

@HeyM1ke HeyM1ke closed this Jan 3, 2021
@HeyM1ke HeyM1ke reopened this Jan 3, 2021
@HeyM1ke
Copy link
Owner

HeyM1ke commented Jan 3, 2021

Oops

@matidfk
Copy link

matidfk commented Jan 4, 2021

By checking if a match is in progress, I meant there might be a way that some script on the client side could use to check if there is one, without needing any info from riots servers. There might not be and thats fine, plus access to local files and sending their data over is quite dodgy with regards to privacy.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants