2
9
Fork
You've already forked FirefoxSidebar
0

Add CSD / Windows Controls #15

Merged
Mr-nUUb merged 2 commits from main into extensions 2022年02月23日 21:26:20 +01:00
Mr-nUUb commented 2022年02月23日 18:31:03 +01:00 (Migrated from github.com)
Copy link

fixes #14

fixes #14

I merged this into a new branch, I am making some changes later today that will help across the board for additional customizations, including yours. This is really great work.

I merged this into a new branch, I am making some changes later today that will help across the board for additional customizations, including yours. This is really great work.

@Mr-nUUb Could you follow the new instructions in the readme (削除) on the 'extensions' branch (削除ここまで) and see if it works for you (削除) (you will need to switch the branch to the extensions branch after completing the clone step, of course)? (削除ここまで) I just merged it with the main branch.

I broke the extension into components (each component has their own file), I added a prefs.css file, and I made it so you can anable/disable tweaks and change settings. I am not having any issues on my side, but I hope that it works the same on yours.

This will also make it so that people can just clone the repo and stay up to date on it.

@Mr-nUUb Could you follow the new instructions in the readme ~~on the 'extensions' branch~~ and see if it works for you ~~(you will need to switch the branch to the extensions branch after completing the clone step, of course)?~~ I just merged it with the main branch. I broke the extension into components (each component has their own file), I added a prefs.css file, and I made it so you can anable/disable tweaks and change settings. I am not having any issues on my side, but I hope that it works the same on yours. This will also make it so that people can just clone the repo and stay up to date on it.

I just merged it with the main branch (on accident, but I'll let it stand for the moment).

I just merged it with the main branch (on accident, but I'll let it stand for the moment).
Xanaxus commented 2022年02月24日 02:13:31 +01:00 (Migrated from github.com)
Copy link

The line no 261 should be changed to 150px IMO as it render like this with 95px:
image
Which is not good according to me

And the line 269 also needs some changes as u can see the window control buttons are not in line with the home button/ prev, pg button, I have kept mine at 10px. Which I guess is due to my custom theme, so I guess a comment will be necessary so that the future users know that this the line which needs to be changed in order to achieve a same plane placement.

This how it looks with my tweaks:
image

The line no 261 should be changed to `150px` IMO as it render like this with `95px`: ![image](https://user-images.githubusercontent.com/53279068/155437377-55c22ccf-e40c-4564-94ed-d0c17cbec27e.png) Which is not good according to me And the line 269 also needs some changes as u can see the window control buttons are not in line with the home button/ prev, pg button, I have kept mine at 10px. Which I guess is due to my custom theme, so I guess a comment will be necessary so that the future users know that this the line which needs to be changed in order to achieve a same plane placement. This how it looks with my tweaks: ![image](https://user-images.githubusercontent.com/53279068/155438360-61d54dc4-b9f7-4230-8fa0-5ae0844ccfc9.png)

Are you using their PR or the changes I have made to this and implemented in the new version of the Repo? (there shouldnt be a line number 261 anymore)

Are you using their PR or the changes I have made to this and implemented in the new version of the Repo? (there shouldnt be a line number 261 anymore)
Xanaxus commented 2022年02月24日 02:22:40 +01:00 (Migrated from github.com)
Copy link

The PR i guess; I just saw the changes in the file changed tab

Will try the new css now didn't see the changes properly

The PR i guess; I just saw the changes in the file changed tab Will try the new css now didn't see the changes properly
Xanaxus commented 2022年02月24日 04:10:52 +01:00 (Migrated from github.com)
Copy link

The pref.css file for the side needs a small change on line 23;
--csd-margin-left: 92px; to --csd-margin-left: 140px;
And the window_contro.css line 25 needs a comment stating that This line handels the position of the window control button on the vertical plane as I needed it for my custom de theme

The pref.css file for the side needs a small change on line 23; `--csd-margin-left: 92px;` to `--csd-margin-left: 140px;` And the window_contro.css line 25 needs a comment stating that `This line handels the position of the window control button on the vertical plane` as I needed it for my custom de theme

That's the beauty of the prefs.css file, the ability to easily change what each system and user needs (140px is far too much for me, but obviously not for you!)

That's the beauty of the prefs.css file, the ability to easily change what each system and user needs (140px is far too much for me, but obviously not for you!)
Xanaxus commented 2022年02月24日 19:15:26 +01:00 (Migrated from github.com)
Copy link

I like it to be 50px but then restore button is overlapped by the prev pg button any idea how to fix it?

I like it to be 50px but then restore button is overlapped by the prev pg button any idea how to fix it?
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
macleod/FirefoxSidebar!15
Reference in a new issue
macleod/FirefoxSidebar
No description provided.
Delete branch "main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?