mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-27 17:15:53 +03:00
[GH-ISSUE #800] What Replaced 'HTTP_TEXT'? #669
Labels
No labels
📶 WiFi
🕸️ HTTP
Branch
DEV Help Wanted
Discussion
Documentation
ESP32
Example
Good First Issue
Hotfix
In Progress
Incomplete
Needs Feeback
Priority
QA
Question
Task
Upstream/Dependancy
bug
duplicate
enhancement
invalid
pull-request
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/WiFiManager#669
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
Originally created by @ghost on GitHub (Jan 8, 2019).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/800
Basic Infos
Hardware
WiFimanager Branch/Release:
Esp8266/Esp32:
Hardware: ESP-12e, esp01, esp25
ESP Core Version: 2.4.0, staging
Description
One used to be able to edit HTTP_TEXT[] PROGMEM inside WiFiManager.h to add text, now with latest version it simply says "WiFiManager" and HTTP_TEXT[] is gone. What has replaced it, as a way to add customized text to the landing page BEFORE the calling /wifi/ form action?
Settings in IDE
Module: Wemos D1
Additional libraries: N/A
@tablatronix commented on GitHub (Jan 8, 2019):
no idea, development version might be easier to modify
@ghost commented on GitHub (Jan 8, 2019):
Development branch moved the entries to strings.h but there is still no HTTP_TEXT[] entry.
It would appear that it comes from WiFiManager.cpp lines 423 to 434, but changing this and rebuilding does not reflect any changes.
@tablatronix commented on GitHub (Jan 8, 2019):
you mean the header?
not sure what you want to change but it is
HTTP_ROOT_MAIN in strings_en.h
but hardcoded in .cpp in the master branch
@tablatronix commented on GitHub (Jan 8, 2019):
you can also set
setCustomHeadElement()but it will not replace just append the to<head>should I add a way to replace the page mast head ?
@ghost commented on GitHub (Jan 8, 2019):
That would be great if it could replace instead of appending.
I'd also request removing the
_apNameand<h3>hardcoded text and simply replacing it with an entry from PROGMEM. I did likeHTTP_TEXT[]as it was named well and obvious what it did.@tablatronix commented on GitHub (Jan 8, 2019):
were you using kentaylors branch maybe ?
@ghost commented on GitHub (Jan 8, 2019):
Initially I was using an old branch that still had HTTP_TEXT, but went an did a full board (to 2.4.2) and library (to 0.14) update and my customized changes didn't apply. Was always with official releases though.
@tablatronix commented on GitHub (Jan 8, 2019):
I don't think so, must have been a PR or something, no way text would have been moved from .h to .cpp and I see no history of HTTP_TEXT
@ghost commented on GitHub (Jan 8, 2019):
You're right, I searched through and never saw HTTP_TEXT either. Not sure, but anyway, if we can make these changes, then it will be easier for users to customize the pages.
@tablatronix commented on GitHub (Jan 8, 2019):
well that header is only on root, do you expect some thing on every page?
@ghost commented on GitHub (Jan 8, 2019):
No, it's fine to simply leave it on root.
I believe
HTTP_HEADandHEAD_ENDis on every page and can be used to "globally" customize every page.@ghost commented on GitHub (Feb 25, 2019):
https://github.com/tzapu/WiFiManager/pull/835