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.

Sign up for GitHub

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

Jump to bottom

Uncomfortable GUI in NEI GT material #8561

Closed
4 tasks done
KamenriderZO opened this issue Sep 19, 2021 · 10 comments
Closed
4 tasks done

Uncomfortable GUI in NEI GT material #8561

KamenriderZO opened this issue Sep 19, 2021 · 10 comments
Assignees
@D-Cysteine
Labels
Mod: NEI Suggestion

Comments

@KamenriderZO
Copy link

Your GTNH Discord Username

GregTechDriver

Your Pack Version

2.1.1.3 dev

Your Proposal

The GUI is not completely shown when in full screen.
like the picture
55$DB5_Q RW_MA@0`N%H~FK

__ S0}8F)AK~I_PP1B74D

Your Goal

optimize the GUI

Your Vision

It's sure

Final Checklist

  • I have searched this issue tracker and there is nothing similar already. Posting on a closed issue saying I like this change please reconsider adding it will prompt us to investigate and reopen it once we confirm your report.
  • I understand this change request may not attract enough attention and thus not be implemented.
  • I understand this change request may be rejected due to other community members think it's inappropriate.
  • I believe this feature would make the pack better.
@KamenriderZO KamenriderZO added Status: Triage Issue awaiting triage. Remove once this issue is processed Suggestion labels Sep 19, 2021
@Pxx500
Copy link
Member

Pxx500 commented Sep 19, 2021

If you want a hotfix then make your GUI smaller in settings.
This should help until someone adds pages

@GTNH-Colen
Copy link
Contributor

Honestly I don't see the big issue here, maybe it should just display an error stating "gui too small" instead but I also don't think we should be supporting super low res monitors or absurd gui sizes.

I have a fairly shit HD monitor and play on normal GUI size, this isn't even close to being an issue on it.

@Pxx500
Copy link
Member

Pxx500 commented Sep 19, 2021

Honestly I don't see the big issue here, maybe it should just display an error stating "gui too small" instead but I also don't think we should be supporting super low res monitors or absurd gui sizes.

I have a fairly shit HD monitor and play on normal GUI size, this isn't even close to being an issue on it.

I thought the same but maybe he has poor eyesight or something like that idk
Normal GUI is the normal one lol

@Prometheus0000
Copy link
Member

I think this was brought up in discord and @D-Cysteine said he was going to try to collapse the tools into each other with cycling through the tier and battery type, reducing 9 to 1 for those ones.

As for not fitting at all, maybe add a scroll bar if it's possible? I think lea gave up on the chisel one because of problems adding one though? So it may not be possible, or really hard to do. I could be remembering wrong though.

@KamenriderZO
Copy link
Author

If ti has a sroll bar will be perfect!

@KamenriderZO
Copy link
Author

Honestly I don't see the big issue here, maybe it should just display an error stating "gui too small" instead but I also don't think we should be supporting super low res monitors or absurd gui sizes.

I have a fairly shit HD monitor and play on normal GUI size, this isn't even close to being an issue on it.

Yes,it made my eyes hurt when i switch it to "normal" .Items are too small ,i can't play the game relaxly

@D-Cysteine
Copy link
Contributor

Yeah, I'm working on fixing this, but it seems like it's a bit tricky to fix so will probably take a while to get a good solution. Will probably either be a scrollbar or extra pages.

@D-Cysteine D-Cysteine self-assigned this Sep 20, 2021
@leagris
Copy link
Contributor

leagris commented Sep 20, 2021

Honestly I don't see the big issue here, maybe it should just display an error stating "gui too small" instead but I also don't think we should be supporting super low res monitors or absurd gui sizes.
I have a fairly shit HD monitor and play on normal GUI size, this isn't even close to being an issue on it.

I thought the same but maybe he has poor eyesight or something like that idk
Normal GUI is the normal one lol

I have poor eye-sight that a birth deffect from my retina, so it cannot be helped by optical correction.

I do need things big enough on screen, especially GUI.

Minecraft natively supports GUI Scale AUTO and all other GUI scales down to small.
If mods cannot layout their GUI in this GUI Scale AUTO and a default smallest Minecraft window size, then mod is broken and the modded GUI need fixing.

You know, I am not asking support for AUTO GUI Scale as a fantasy for my own comfort.

  1. Other GUI scales are not an option to me.
  2. It is not even a fantasy selfish request from an annoying lady with a very low eye sight disability, and who could consider moving to other free time occupation than coding, modding and sometimes playing modded with GTNH.
  3. This is fixing a broken GUI layout.

Period!

I am pretty sure @D-Cysteine will find a way to fix this GUI layout issue, because He is a smart and capable modder, and his NEI add-on with Gregtech materials is absolutely clever and useful.

D-Cysteine added a commit to D-Cysteine/nei-custom-diagram that referenced this issue Oct 1, 2021
@D-Cysteine
v1.0.0
ba9b666 
 * Implemented scrolling with mouse wheel
   * Can scroll pages, diagrams, and cycle components
   * Fix for GTNewHorizons/GT-New-Horizons-Modpack#8561
 * Revamped GregTechOreProcessing diagram layout
   * Merged thermal centrifuge layouts to free up space
   * Added chemical bath recipes to diagram
   * Thanks to bombcar for the layout idea!
 * Fixed hotkey integration with NEI; should now work with 'T' and 'Y'
 * Can now view cycle components in tooltip
 * Added config option to hide empty diagrams.
 * Added diagram group description, for showing in config comments.
 * Added separate mod check support for GT 5 vs. GT 6.
   * Includes large rename of packages and variables to include GT version.
   * Fixes crash in #1
 * Various small diagram improvements:
   * Added GregTechOrePrefixes diagram
   * Added bars to GregTechMaterialParts diagram
   * Properly sorted tools by EU capacity in GregTechMaterialTools diagram
   * Added "all diagrams" button to ForgeFluidContainers
   * Removed numbering from DebugRuler due to truncation
 * Now needs NEI v2.1.13-GTNH or later
 * Significant restructuring and improvements to code
D-Cysteine added a commit to D-Cysteine/nei-custom-diagram that referenced this issue Oct 1, 2021
@D-Cysteine
v1.0.0
4472152 
 * Implemented scrolling with mouse wheel
   * Can scroll pages, diagrams, and cycle components
   * Fix for GTNewHorizons/GT-New-Horizons-Modpack#8561
 * Revamped GregTechOreProcessing diagram layout
   * Merged thermal centrifuge layouts to free up space
   * Added chemical bath recipes to diagram
   * Thanks to bombcar for the layout idea!
 * Fixed hotkey integration with NEI; should now work with 'T' and 'Y'
 * Can now view cycle components in tooltip
 * Added config option to hide empty diagrams.
 * Added diagram group description, for showing in config comments.
 * Added separate mod check support for GT 5 vs. GT 6.
   * Includes large rename of packages and variables to include GT version.
   * Fixes crash in #1
 * Various small diagram improvements:
   * Added GregTechOrePrefixes diagram
   * Added bars to GregTechMaterialParts diagram
   * Properly sorted tools by EU capacity in GregTechMaterialTools diagram
   * Added "all diagrams" button to ForgeFluidContainers
   * Removed numbering from DebugRuler due to truncation
 * Now needs NEI v2.1.13-GTNH or later
 * Significant restructuring and improvements to code
@D-Cysteine
Copy link
Contributor

Sorry for the long wait, but the scrolling feature is finally done! Please give it a try and let me know if it works for you, or if it needs changes. Right now, the scrolling works purely off of the mouse wheel. Let me know if that's an issue for you.

Get the latest release here:

@KamenriderZO
Copy link
Author

Sorry for the long wait, but the scrolling feature is finally done! Please give it a try and let me know if it works for you, or if it needs changes. Right now, the scrolling works purely off of the mouse wheel. Let me know if that's an issue for you.

Get the latest release here:

excellent!The scroll bar works fine.
$0WVSK0ANFHYOS$ON16`3

RJG(XD$ T6 VGYHDQG(LK)G

@Prometheus0000 Prometheus0000 added Status: FixedInDev Mod: NEI and removed Status: Triage Issue awaiting triage. Remove once this issue is processed labels Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@D-Cysteine D-Cysteine

Labels
Mod: NEI Suggestion
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@leagris @Prometheus0000 @D-Cysteine @GTNH-Colen @KamenriderZO @Pxx500